Skip to content

Conversation

@SamLau95
Copy link
Contributor

@SamLau95 SamLau95 commented Sep 4, 2015

Also makes Table.hist and Table.barh throw a more helpful error message when the
Table contains invalid values (np.flexible, which includes strings).
Before, the method raised an error about not being able to operate on
"flexible types", which is not extremely helpful without knowledge of
numpy types.

Starts to address #50.

Travis tests are failing because of a long-standing issue with the date formatter.

Also makes `Table.hist` throw a more helpful error message when the
Table contains invalid values (`np.flexible`, which includes strings).
Before, the method raised an error about not being able to operate on
"flexible types", which is not extremely helpful without knowledge of
numpy types.
@SamLau95 SamLau95 changed the title Add more documentation to Table.hist Add more documentation to hist and barh Sep 5, 2015
As well as raising a ValueError with a helpful error message if
we try to graph non-numerical values.
@SamLau95 SamLau95 force-pushed the hist-docstring branch 2 times, most recently from b466839 to 9218d34 Compare September 5, 2015 08:28
@SamLau95
Copy link
Contributor Author

SamLau95 commented Sep 9, 2015

@papajohn @alvinwan Either one of you wanna give this a review?

Copy link
Contributor

Choose a reason for hiding this comment

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

numerical

@papajohn
Copy link
Contributor

papajohn commented Sep 9, 2015

Yes, that's much improved!

papajohn added a commit that referenced this pull request Sep 9, 2015
Add more documentation to hist and barh
@papajohn papajohn merged commit 0bbbad0 into master Sep 9, 2015
@papajohn papajohn deleted the hist-docstring branch September 9, 2015 03:56
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