Skip to content

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