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 — Keep CSV reader state coherent across callbacks by youknowdot · Pull Request #9449 · RustPython/RustPython Conversation
Contributor
SummaryKeep CSV reader state in a shared native parser while releasing short payload borrows before callbacks, reference drops and Python allocation. Includes the first consumer of the interpreter-bound PyOperation facade, the existing float-error helper extraction, and the whole canonical CSV free-threading test file. This is a functionality-based extraction from #9439, based on main All included Lib files were regenerated with actual Validation and limitsEarlier aggregate: 24 serial CSV oracles and 11 ordinary canonical methods passed on a different combined executable. They are historical evidence only; fresh standalone hosted CI is still required. No new local concurrency test was run. Fresh standalone check and strict Clippy (
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactionsExtract CSV reader synchronization with its PyOperation facade and float-error helper from RustPython#9439. Include the whole canonical free-threading CSV test regenerated by scripts/update_lib from CPython d220409a13b77de7a6c30abe16ea90db01528edc. Assisted-by: OpenAI Codex: model version unavailable
|
There was an error while loading. Please reload this page.
|
Caution Review failedThe pull request is closed. Finishing Touches
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.
📦 Library DependenciesThe following Lib/ modules were modified. Here are their dependencies: [ ] test: cpython/Lib/test/test_free_threading dependencies: dependent tests: (no tests depend on free_threading) Legend:
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Merging this PR will not alter performance
Comparing Footnotes
|
Sorry, something went wrong.
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 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.