8000 BUG: None comparison deprecation does not affect scalars by seberg · Pull Request #4910 · numpy/numpy · GitHub
[go: up one dir, main page]

Skip to content

BUG: None comparison deprecation does not affect scalars #4910

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

Closed
wants to merge 1 commit into from

Conversation

seberg
Copy link
Member
@seberg seberg commented Jul 28, 2014

This circumvents the None warnings for scalar comparisons.
Unfortunatly datetime NaT can in some cases evaluate equal
to None because NaT.item() is None.

Closes gh-4894

@seberg
Copy link
Member Author
seberg commented Jul 28, 2014

Hmmmm, I tried git rebase --onto $(git merge-base master maintenance/1.9.x) HEAD^ but now things seem gone...

@seberg
Copy link
Member Author
seberg commented Jul 28, 2014

Hmm, or maybe I just did something wrong before the rebase, quite probable.

This circumvents the None warnings for scalar comparisons.
Unfortunatly datetime NaT *can* in some cases evaluate equal
to None because NaT.item() is None.

Closes numpygh-4894
@seberg
Copy link
Member Author
seberg commented Jul 28, 2014

Hmmm, seemed to get some merge trouble in the deprecation tests, so didn't rebase it now.

@juliantaylor
Copy link
Contributor

rebasing works fine for me, thanks I'll push it directly.

juliantaylor added a commit that referenced this pull request Jul 28, 2014
BUG: None comparison deprecation does not affect scalars
juliantaylor added a commit that referenced this pull request Jul 28, 2014
BUG: None comparison deprecation does not affect scalars
@juliantaylor
Copy link
Contributor

pushed as 8624924 thanks

@seberg
Copy link
Member Author
seberg commented Jul 28, 2014

Last thing I had tried was cherry picking the commit onto an empty rebased branch. I suppose that isn't the same thing.

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.

1.9.0b1: Strange FutureWarning for object array containing np.float32
2 participants
0