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

FEAT: Add DataFramePlotData. #367

Merged
merged 8 commits into from
Jul 19, 2017
Merged

FEAT: Add DataFramePlotData. #367

merged 8 commits into from
Jul 19, 2017

Conversation

cfarrow
Copy link
Contributor

@cfarrow cfarrow commented Jul 15, 2017

This is for you, @jonathanrocher. It needs a bit more testing around 'index' and an example.

@jwiggins
Copy link
Member

You should add pandas to the CI requirements

@cfarrow
Copy link
Contributor Author

cfarrow commented Jul 19, 2017

Thanks for the tip. it took a few tries to get it right. :)

Code + example + tests (passing) = ready for review and merge

""" A PlotData implementation class that handles a DataFrame.

By default, it doesn't allow its input data to be modified by downstream
Chaco components or interactors. The index is availble as data unless there
Copy link
Contributor

Choose a reason for hiding this comment

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

typo "available"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I cannot spell that word right. I need to either stop using it or spell check every time I do. :D

@cfarrow
Copy link
Contributor Author

cfarrow commented Jul 19, 2017

Any other suggestions?

@jvkersch
Copy link
Contributor

Looks great to me! Esp. the demo.

@jvkersch jvkersch merged commit ca4ca5d into master Jul 19, 2017
@jvkersch jvkersch deleted the feature/data_frame_plot_data branch July 19, 2017 20:08
@cfarrow
Copy link
Contributor Author

cfarrow commented Jul 19, 2017

Thanks!

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.

3 participants