Skip to content

Merge upstream - #1

Open
raldridge11 wants to merge 125 commits into
raldridge11:masterfrom
lava:master
Open

Merge upstream#1
raldridge11 wants to merge 125 commits into
raldridge11:masterfrom
lava:master

Conversation

@raldridge11

Copy link
Copy Markdown
Owner

Resolve conflicts, and merge.

flyvbee and others added 30 commits October 25, 2017 19:18
* Add support for stem and xkcd
* Add README for xkcd
…f the Python version 3 and void if the Python version is 2.
At the moment, the errorbar() function takes an optional parameter s,
which is presumably supposed to correspond to the same parameter for
e.g. the plot function. The only problem is that a) this argument is
unused in the function and b) matplotlib's errorbar() doesn't take a
parameter like this anyway.

I've replaced the s argument with an optional keywords argument, as for
other functions, which allows for setting colour, marker style etc.
pf and others added 30 commits June 17, 2020 18:45
…ple of them to work. In the future matplotlibcpp should use map<string,any> instead of map<string,string> for kwargs. That should be a separate PR
The Python interpreter can be constructed by any thread while the destructor is always the main thread. This can lead in some errors with the Python side of things, where some objects want the thread which constructed them to also destroy them. By adding this 'kill' function, a developer can use your library (btw gj! 👍 ) with `std::thread` in order to create a plot async -aka without blocking the main thread- and then close the plot and 'kill' Python interpreter afterwards, without waiting the program to end or having destructor errors.

Doing this in order to showcase this more. Small change but not that polished. Let me know about your opinion first and we can fix it 😃.
Thank you for your time and this great library ❤️
Memory is allocated with new and delete is never called. Use a std::array instead, so no memory will be allocated.
After the switch to cmake, we don't need the hand-written Makefile
nor the contrib/ version of the cmake file.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.