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.

Sign up for GitHub

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

Jump to bottom

Stop treating inf/-inf as missing #1919

Closed
wesm opened this issue Sep 16, 2012 · 4 comments
Closed

Stop treating inf/-inf as missing #1919

wesm opened this issue Sep 16, 2012 · 4 comments
Milestone
0.10

Comments

@wesm
Copy link
Member

wesm commented Sep 16, 2012

No description provided.

@wesm
Copy link
Member Author

wesm commented Sep 24, 2012

This could be made configurable

@jseabold
Copy link
Contributor

jseabold commented Oct 1, 2012

Do you plan to change the behavior of isnull to achieve this? We are using internally in statsmodels now for missing data handling, so I just want to be sure. If so, maybe use a keyword argument to set what's missing / what's not.

@wesm wesm mentioned this issue Oct 6, 2012
Closed
aflaxman added a commit to aflaxman/pandas that referenced this issue Oct 8, 2012
@aflaxman
ENH: make None/NaN different from INF/-INF (GH pandas-dev#1919)
d73b033
@aflaxman
Copy link
Contributor

aflaxman commented Oct 8, 2012

I have a branch with this implemented, related to #2026. It does not yet have a way to configure inf-as-nan on/off: aflaxman@fe12c8a

@aflaxman aflaxman mentioned this issue Oct 8, 2012
Closed
aflaxman added a commit to aflaxman/pandas that referenced this issue Oct 9, 2012
@aflaxman
ENH: add method use_inf_as_null to core.common (GH pandas-dev#1919)
7b79b66
@aflaxman
Copy link
Contributor

aflaxman commented Oct 9, 2012

I have completed an implementation of a way to turn inf-as-nan on and off. Patches welcome...
aflaxman@7b79b66

I will send a pull request for this now, but let me know if there is additional work necessary and I'll see what I can do.

@aflaxman aflaxman mentioned this issue Oct 9, 2012
Merged
@wesm wesm closed this as completed Dec 2, 2012
@ghost ghost mentioned this issue Dec 12, 2012
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.10
Development

No branches or pull requests
3 participants
@aflaxman @jseabold @wesm