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

Cfd plot fix #2323

Closed
wants to merge 3 commits into from
Closed

Cfd plot fix #2323

wants to merge 3 commits into from

Conversation

jbbe
Copy link

@jbbe jbbe commented Jun 21, 2019

The plot function in ConditionalFreqDist was throwing an ImportError
because it was trying to import 'plt' from matplotlib which does not
exist. Additionally the methods called were still the methods of
they pylab submodule and I updated those calls to those of the pyplot
submodule. Additionally the if v in self key check would not pass
with valid input so I removed it and the function now plots correctly.

jbbe added 3 commits June 21, 2019 16:01
The plot function in ConditionalFreqDist was throwing an ImportError
because it was trying to import 'plt' from matplotlib which does not
exist. Additionally the methods called were still the methods of
they pylab submodule and I updated those calls to those of the pyplot
submodule. Additionally the if v in self key check would not pass
with valid input so I removed it and the function now plots correctly.
Was calling ax.set_xticks which was not displaying the correct labels
so I changed it to use the pyplot.xticks method which works correctly
with the input that was being given to set_xticks.
@stevenbird
Copy link
Member

Thanks @jbbe.
Resolved in #2319

@stevenbird stevenbird closed this Jul 4, 2019
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.

None yet

2 participants