-
Notifications
You must be signed in to change notification settings - Fork 1.2k
ENH use variadic templates args in scatter and figure #166
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
Bidibulke
wants to merge
9
commits into
lava:master
Choose a base branch
from
Bidibulke:merge/var_temp_args
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
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
TST * adding some unit tests using Catch2 framework * 6 test cases with 91 assertions. * normally there is a test for each function in matplotlibcpp.h * `class Plot` is not tested MISC According to numpy documentation: * BLD change related to building and installing (examples, module, …) * ENH enhancement * TST addition or modification of tests * ...
commit 9047863bb0c1a34bc6473c2dadee720073db631e Author: Claire Guilbaud <[email protected]> Date: Thu Mar 26 15:46:34 2020 +0100 @enh figure with variadic template args commit a8a3dd018c14a5e5c98b084f566865ba936c4fec Author: Claire Guilbaud <[email protected]> Date: Thu Mar 26 14:55:36 2020 +0100 @enh close commit 6440609657de0f47b6fd4c1a8c213704f2810a4f Author: Claire Guilbaud <[email protected]> Date: Thu Mar 26 13:41:18 2020 +0100 @enh generic form for scatter
9ae729d
to
06425a4
Compare
commit b9bd9d7488d0109f58d195c3964631635614e5e1 Author: Claire Guilbaud <[email protected]> Date: Fri May 29 15:29:00 2020 +0200 ENH quiver extension / streamplot added
ENH quiver * adding ``quiver(x, y, u, w, color)`` * adding support of variadic template args in ``quiver`` ENH streamplot * adding ``streamplot(x, y, u, w)`` with variadic template args * adding ``analyze_key_value`` for ``vector<vector<double>>`` TST * new C++ Catch2 tests for new functionalities
I added streamplot function (still with variadic template args). I also extended quiver to accept a vector color as 5th arguments (in the generic form), and new tests for those functionalities. |
Interpreter has to be initialized otherwise PyLong_FromLong returns a not null object, but null inside ...
ENH: * New functions: `plot_surface`, `imshow`, `stem` * Added new example * Tests with Catch2 updated MAINT: * replace __function by detail::function for intern functions like for imshow BUG FIX PyLong_FromLong in analyse_keywords * Interpreter has to be initialized otherwise PyLong_FromLong returns a not null object, but null inside ...
|
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.
In order to avoid adding other declaration function for the same functionnality with other arguments, I had the possibility to pass extra-arguments in a tuple of variadic templates.
The kwargs of matplotlib functions are accessible using a tuple of pair (string, T). T can be :
The syntax is not heavy, and if other types are needed, just implement a function such as
Some examples:
streamplot
scatter