Skip to content
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

Save plots as pickled figures #23

Closed
wants to merge 6 commits into from
Closed

Conversation

joniemi
Copy link

@joniemi joniemi commented Jun 3, 2020

When testing complex data, I find it helpful to navigate the produced figures using pyplot's interactive GUI interface. This pull request implements this feature by giving the option of storing figures in pickle files which can be loaded later for inspection. The option is triggered if a user defines an output filename with a ".pickle" suffix.

Would this feature be useful for a wider public?

I have not signed my name to the contributor list yet.

@hunse
Copy link
Collaborator

hunse commented Jul 22, 2020

Hi @joniemi. Thanks for your contribution! I believe this could be useful to others.

Can you please sign the contributor's agreement by making a PR to add your name to this list (the source file is here)?

@hunse
Copy link
Collaborator

hunse commented Jul 22, 2020

@joniemi Sorry, never mind! I was incorrect. Since this repo is under an open-source license, you don't need to sign the contributor's agreement.

Copy link
Collaborator

@hunse hunse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've merged this with some minor changes (no changes to the functionality, though).

@joniemi
Copy link
Author

joniemi commented Aug 3, 2020

Thanks @hunse and @tbekolay! Looking forward to the next release. :)

@drasmuss
Copy link
Member

pytest-plt 1.1.0 has now been released (containing this feature), thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants