GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||||||||||||||||
GeezerVison is showing github.com — math.log: handle large integer bases and error order by youknowdot · Pull Request #9019 · RustPython/RustPython Conversation
Contributor
SummarySplit the Python 3.14-compatible Use the existing natural-log conversion for both operands, evaluating Keep Extend the existing math snippet with large bases, base one, conversion ordering, integer subclasses, and conversion-error cases. No canonical standard-library assertions change. Validation
Local runtime validation is limited to the focused snippet, observations, and canonical tests above. Broad Rust test targets were compile-checked, not executed. References: CPython 3.14 log conversion and evaluation order, float division. AI assistance
OpenAI Codex assisted with auditing the split, implementation, regression coverage, and validation. The runtime does not expose an exact model identifier.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactionsUse the natural logarithm path for both operands, evaluating x first. Preserve large integer bases and raise ZeroDivisionError for base one while retaining noninteger conversion errors. Split the Python 3.14-compatible behavior from RustPython#8954 without its additional __index__ fallback or changes to log2 and log10. Assisted-by: OpenAI Codex:model identifier unavailable
|
There was an error while loading. Please reload this page.
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configuration
📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Here are some automated review suggestions for this pull request.
Reviewed commit: 9d17669334
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Sorry, something went wrong.
There was an error while loading. Please reload this page.
| # assert math.log(1.1111) == math.log(Conversible()) | ||
|
|
||
| # A logarithm can fit in a float even when its integer base cannot. | ||
| assert math.isclose(math.log(2**4000, 2**2000), 2.0) |
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Remove the forbidden test additions
This block adds new assertions, helper classes, and test data to extra_tests/snippets/stdlib_math.py, but the repository's critical test-code rules forbid modifying test assertions, logic, or data and state that the only acceptable test-file changes are adding or removing expected-failure decorators. Remove this new test logic from the patch (or obtain a documented policy exception) before merging.
AGENTS.md reference: AGENTS.md:L273-L279
Useful? React with 👍 / 👎.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.