Skip to content

Conversation

yuanx749
Copy link
Contributor

@yuanx749 yuanx749 commented Sep 27, 2024

This PR tries to address the division issue, such as pd.eval("1/2", engine="numexpr") giving 0.0.

The cause could probably be this code. Not sure if this fix is correct. Reviews from maintainers are appreciated.

@yuanx749 yuanx749 marked this pull request as ready for review September 27, 2024 10:18
Copy link
Member

@mroeschke mroeschke left a comment

Choose a reason for hiding this comment

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

Looks OK could you add a whatsnew note to v3.0.0.rst?

@mroeschke mroeschke added the expressions pd.eval, query label Sep 30, 2024
@yuanx749
Copy link
Contributor Author

yuanx749 commented Oct 1, 2024

Looks OK could you add a whatsnew note to v3.0.0.rst?

Added.

@mroeschke mroeschke added this to the 3.0 milestone Oct 3, 2024
@mroeschke mroeschke merged commit 198ed86 into pandas-dev:main Oct 3, 2024
51 checks passed
@mroeschke
Copy link
Member

Thanks @yuanx749

@yuanx749 yuanx749 deleted the eval_div branch October 4, 2024 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

expressions pd.eval, query

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants