-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Make median! type stable for small float types #29902
Conversation
I have an alternative fix that returns the first NaN value, preserving the payload of the specific NaN. Not sure if we care about that, but it is how arithmetic operations behave. |
I don't have strong feelings here but on my machine there seems to be a time penalty for using |
Yeah, probably doesn't matter. Unfortunate that |
Fixed #29900