We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Option<usize>
Thought it would be useful to return None when there are NaNs in the array.
None
But this seems less useful / necessary than I thought as np.argmin returns (just as the current code base) the index of the NaN in the array ⬇️
np.argmin
>>> np.array([1, 2, np.nan, 4], dtype=float).argmin() 2
So the only use-case for returning None would be;
The second use-case seems also less useful / necessary as np.argmin returns (just as the current code base) 0 ⬇️
>>> np.array([np.nan, np.nan], dtype=float).argmin() 0
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Thought it would be useful to return
None
when there are NaNs in the array.But this seems less useful / necessary than I thought as
np.argmin
returns (just as the current code base) the index of the NaN in the array ⬇️So the only use-case for returning
None
would be;The second use-case seems also less useful / necessary as
np.argmin
returns (just as the current code base) 0 ⬇️The text was updated successfully, but these errors were encountered: