[FIRST] plotlyupdate.m #27
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
[CLEAN] space between outputted "verbose" descriptions
.gitignore Outdated
Member
There was a problem hiding this comment.
if this is a folder containing test results, we might actually want to keep it in the repo, so that we can check test diffs on PRs (in the absence of something more automated)
Member
| super awesome! |
[UPDATE] wrapper parseLatex [UPDATE] version number
Member
| So, how do you feel about issuing warnings if they are using an out-of-date version?
|
Member
There was a problem hiding this comment.
oh, clever. if they're not equal, then local is outdated. much nicer than comparing 2-decimal placing numbers!
[FIX] plotlysetup to use key/value input structure for varargin
[ADD] plotlyclean to plotlyupdate
Conflicts: plotlysetup.m
…rsion_update_dev
[ADD] wrong input property name catch
[FIX] location of plotly instances check
Member Author
| @chriddyp Latest changes address your suggestions! tests are on the development branch if you want to run them : ) |
Member
| sweet, this is looking good! |
Member
| did you update the version number? |
Member Author
| Not yet - i'll bump the version before the merge! |
[ADD] permission message helper function
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
@chriddyp wanna look this over when you get a chance.
plotlyupdate
plotlyupdate('verbose')
ploltyupdate('verbose','nocheck')