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

Switch the backend ot matplotlib #142

Closed
wants to merge 1 commit into from
Closed

Conversation

tsnouidui
Copy link
Member

This closes #141.

@tsnouidui tsnouidui requested a review from dhblum June 13, 2018 00:20
@dhblum
Copy link
Collaborator

dhblum commented Jun 13, 2018

Thanks @tsnouidui. This fix essentially switches the figure to be generated on the back-end only, and not in an interactive mode. What this means is that something like plt.show() no longer works to show a plot, but plt.savefig() still works. I'm therefore hesitant to make this a permanent fix in MPCPy, as it will change the figure settings for a user without knowledge.

I'm wondering if there is a way to make it a switch that can be enabled when running in a Docker?

Copy link
Collaborator

@dhblum dhblum left a comment

Choose a reason for hiding this comment

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

See comment on pull request thread.

@dhblum
Copy link
Collaborator

dhblum commented Jun 13, 2018

The proposed solution will not work. Closing this pull request.

@dhblum dhblum closed this Jun 13, 2018
@dhblum dhblum deleted the issue141_display branch June 13, 2018 21:03
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.

Running MPCPy with plots in docker without display
2 participants