Skip to content

REGR: fillna not filling NaNs after pivot without explicitly listing pivot values on 1.1.x #36668

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

Conversation

simonjayhawkins
Copy link
Member

@simonjayhawkins simonjayhawkins commented Sep 26, 2020

yet another alternative to fix directly on 1.1.x pending further discussion in #34407 on how to resolve the issues on master.

cc @jorisvandenbossche @jbrockmendel

This won't close issue until fix on master. could copy test to master with xfail (and would need to sync release notes if we merge this direct to 1.1.x)

@simonjayhawkins simonjayhawkins added Internals Related to non-user accessible pandas implementation Missing-data np.nan, pd.NaT, pd.NA, dropna, isnull, interpolate Regression Functionality that used to work in a prior pandas version labels Sep 26, 2020
@simonjayhawkins simonjayhawkins added this to the 1.1.3 milestone Sep 26, 2020
@simonjayhawkins
Copy link
Member Author

travis environment setup failure unrelated

@simonjayhawkins simonjayhawkins marked this pull request as ready for review September 26, 2020 14:57
@pep8speaks
Copy link

pep8speaks commented Sep 26, 2020

Hello @simonjayhawkins! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2020-09-26 15:05:56 UTC

@jorisvandenbossche
Copy link
Member

Fine for me to merge this now in 1.1.x, and have another version applied to master with the test xfailed

@jbrockmendel
Copy link
Member

I'll try to find the underlying problem so this is unnecessary. In the interim, this is fine by me.

@simonjayhawkins
Copy link
Member Author

may need more discussion.

I can't block by requesting changes on my own PR, moving to 1.1.4 so release note needs to be moved. closing for now instead. Will re-open after 1.1.3 release.

I've started the final release readiness checks. merging now would require going back to square one.

@jorisvandenbossche
Copy link
Member

Why not include this in the release? This PR is ready it seems (small fix, tests pass)

@simonjayhawkins
Copy link
Member Author

merging directly to 1.1.x can be a bit messy. If there's a better way #36668 (comment) we can do that for 1.1.4

If this issue still persists when we are ready for 1.1.4 release we could merge this instead?

@jorisvandenbossche
Copy link
Member

Sounds good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Internals Related to non-user accessible pandas implementation Missing-data np.nan, pd.NaT, pd.NA, dropna, isnull, interpolate Regression Functionality that used to work in a prior pandas version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants