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

TST: matplotlib 2.0 fix in log limits for barplot (GH14808) #14957

Merged
merged 2 commits into from
Dec 24, 2016

Conversation

jorisvandenbossche
Copy link
Member

Closes #14808

@jorisvandenbossche jorisvandenbossche added Testing pandas testing functions or related to the test suite Visualization plotting labels Dec 22, 2016
@jorisvandenbossche jorisvandenbossche added this to the 0.19.2 milestone Dec 22, 2016
@TomAugspurger
Copy link
Contributor

👍 when the tests pass

@codecov-io
Copy link

codecov-io commented Dec 22, 2016

Current coverage is 84.66% (diff: 100%)

Merging #14957 into master will increase coverage by <.01%

@@             master     #14957   diff @@
==========================================
  Files           144        144          
  Lines         51030      51047    +17   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits          43198      43217    +19   
+ Misses         7832       7830     -2   
  Partials          0          0          

Powered by Codecov. Last update 8b497e4...1f1a80d

@jorisvandenbossche jorisvandenbossche merged commit f293d62 into pandas-dev:master Dec 24, 2016
@jorisvandenbossche
Copy link
Member Author

@TomAugspurger had to change it a bit further (the build in allowed failures for matplotlib 2.0cr2 was stll failing). Merged now, but I am wondering if we should actually test those 'implementation' details of matplotlib, and not just that the plot works.

jorisvandenbossche added a commit to jorisvandenbossche/pandas that referenced this pull request Dec 24, 2016
@TomAugspurger
Copy link
Contributor

I agree that the test isn't all that useful.

ShaharBental pushed a commit to ShaharBental/pandas that referenced this pull request Dec 26, 2016
yarikoptic added a commit to neurodebian/pandas that referenced this pull request Jan 3, 2017
Version 0.19.2

* tag 'v0.19.2': (78 commits)
  RLS: v0.19.2
  DOC: update release notes for 0.19.2
  TST: skip gbq upload test as flakey
  DOC: clean-up v0.19.2 whatsnew
  DOC: update Pandas Cheat Sheet (GH13202)
  DOC: Pandas Cheat Sheet
  TST: matplotlib 2.0 fix in log limits for barplot (GH14808) (pandas-dev#14957)
  flake8 fix import
  Remove test - from 0.20.0 PR slipped in
  PERF: fix getitem unique_check / initialization issue
  cache and remove boxing (pandas-dev#14931)
  CLN: Resubmit of GH14700.  Fixes GH14554.  Errors other than Indexing…
  Clean up construction of Series with dictionary and datetime index
  BUG: .fillna() for datetime64 with tz is passing thru floats
  BUG: Patch read_csv NA values behaviour
  ENH: merge_asof() has type specializations and can take multiple 'by' parameters (pandas-dev#13936)
  [Backport pandas-dev#14886] BUG: regression in DataFrame.combine_first with integer columns (GH14687) (pandas-dev#14886)
  Fixed KDE Plot to drop the missing values (pandas-dev#14820)
  ENH: merge_asof() has left_index/right_index and left_by/right_by (pandas-dev#14253) (pandas-dev#14531)
  TST: correct url for test file on s3 (xref pandas-dev#14587)
  ...
yarikoptic added a commit to neurodebian/pandas that referenced this pull request Jan 3, 2017
* releases: (78 commits)
  RLS: v0.19.2
  DOC: update release notes for 0.19.2
  TST: skip gbq upload test as flakey
  DOC: clean-up v0.19.2 whatsnew
  DOC: update Pandas Cheat Sheet (GH13202)
  DOC: Pandas Cheat Sheet
  TST: matplotlib 2.0 fix in log limits for barplot (GH14808) (pandas-dev#14957)
  flake8 fix import
  Remove test - from 0.20.0 PR slipped in
  PERF: fix getitem unique_check / initialization issue
  cache and remove boxing (pandas-dev#14931)
  CLN: Resubmit of GH14700.  Fixes GH14554.  Errors other than Indexing…
  Clean up construction of Series with dictionary and datetime index
  BUG: .fillna() for datetime64 with tz is passing thru floats
  BUG: Patch read_csv NA values behaviour
  ENH: merge_asof() has type specializations and can take multiple 'by' parameters (pandas-dev#13936)
  [Backport pandas-dev#14886] BUG: regression in DataFrame.combine_first with integer columns (GH14687) (pandas-dev#14886)
  Fixed KDE Plot to drop the missing values (pandas-dev#14820)
  ENH: merge_asof() has left_index/right_index and left_by/right_by (pandas-dev#14253) (pandas-dev#14531)
  TST: correct url for test file on s3 (xref pandas-dev#14587)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Testing pandas testing functions or related to the test suite Visualization plotting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants