Skip to content

BUG: Allow plotting boolean values #27665

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

Merged
merged 17 commits into from
Aug 12, 2019

Conversation

charlesdong1991
Copy link
Member

Copy link
Contributor

@TomAugspurger TomAugspurger left a comment

Choose a reason for hiding this comment

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

Also need a release note in 1.0.0.rst.

@TomAugspurger TomAugspurger added this to the 1.0 milestone Jul 31, 2019
@TomAugspurger TomAugspurger added the Visualization plotting label Jul 31, 2019
@charlesdong1991
Copy link
Member Author

somehow even after i merge with master, this still fails... but in the meantime i see other PRs passing all checks, what would be the reason that fails this one? @jreback @TomAugspurger help or explanation will be very appreciated!

@TomAugspurger
Copy link
Contributor

What versions of pytest & pytest-xdist are present in that failing build?

@charlesdong1991
Copy link
Member Author

I suppose it's pytest-5.0.1 and xdist-1.28.0? @TomAugspurger

Screen Shot 2019-08-07 at 11 20 09 PM

@TomAugspurger
Copy link
Contributor

TomAugspurger commented Aug 7, 2019 via email

@charlesdong1991
Copy link
Member Author

charlesdong1991 commented Aug 8, 2019

still fails, should pytest and xdist version also be changed in ci/deps/azure-37-locale.yaml and ci/deps/travis-36-slow.yaml? since this check fails in my case. @TomAugspurger I checked last PR 27809 in which azure-36-locale was changed.

@TomAugspurger
Copy link
Contributor

Hmm perhaps. I'm surprised to see that, but maybe defaults doesn't have the newest pytest-xdist version.

@charlesdong1991
Copy link
Member Author

ok, i just try updating the version of those two packages to see if the pr could pass all checks this time, if so, i will open another pr to separate this issue @TomAugspurger

@charlesdong1991
Copy link
Member Author

one more check is passed... by this travis one still could not... i think i will leave this here for now... I don't have much knowledge about CI, and really have no idea what could lead to this @TomAugspurger

@TomAugspurger
Copy link
Contributor

TomAugspurger commented Aug 8, 2019 via email

@charlesdong1991
Copy link
Member Author

charlesdong1991 commented Aug 9, 2019

@TomAugspurger it used to pass all the checks if the versions were changed to newer ones for pytest and xdist, but if I keep it as is, then the Windows py37_np141 fails again. Not sure if this is the reason fails the PR, or should I open another PR to fix this version issue?

@TomAugspurger
Copy link
Contributor

#27845 is trying to get them all.

@charlesdong1991
Copy link
Member Author

charlesdong1991 commented Aug 12, 2019

it finally passes all checks @TomAugspurger thanks for fixing CI!! ^^

@TomAugspurger TomAugspurger merged commit 6afa2ad into pandas-dev:master Aug 12, 2019
@TomAugspurger
Copy link
Contributor

Thanks @charlesdong1991. Sorry about all the CI failures!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Plotting boolean data fails
4 participants