Skip to content

Fix edge case of math.pow(0.0, -inf), math.pow(-0.0, -inf)#4152

Merged
youknowone merged 1 commit intoRustPython:mainfrom
jopemachine:fix-math-pow
Sep 12, 2022
Merged

Fix edge case of math.pow(0.0, -inf), math.pow(-0.0, -inf)#4152
youknowone merged 1 commit intoRustPython:mainfrom
jopemachine:fix-math-pow

Conversation

@jopemachine
Copy link
Copy Markdown
Contributor

@jopemachine jopemachine commented Sep 9, 2022

Python documentation says The special cases pow(0.0, -inf) and pow(-0.0, -inf) were changed to return inf instead of raising ValueError, for consistency with IEEE 754

Related links

Copy link
Copy Markdown
Member

@youknowone youknowone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice catch!

@youknowone youknowone merged commit a719955 into RustPython:main Sep 12, 2022
@youknowone youknowone added the z-ca-2022 Tag to track contrubution-academy 2022 label Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

z-ca-2022 Tag to track contrubution-academy 2022

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants