| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
This reduces the import time of the `csv` module by up to five times, by importing `re` on demand. In particular, the `re` module is no more implicitly exposed as `csv.re`.
There was a problem hiding this comment.
LGTM
Sorry, something went wrong.
|
In a few hours, I will leave until Friday, but I'd like Adam or Hugo to review my English. Feel free to merge this PR and freely update it (otherwise, I'll do it once I'm back). |
Sorry, something went wrong.
There was a problem hiding this comment.
Looks good, thanks!
Sorry, something went wrong.
This reduces the import time of the `csv` module by up to five times, by importing `re` on demand. In particular, the `re` module is no more implicitly exposed as `csv.re`.
| Back | FazBrowse Home | New Git URL |
Benchmarks are for a release non-PGO build (PR) against a PGO build (main). This will be the last PR for that issue.
Note
Considering that csv is used in many applications, I think this improvement makes sense (especially for data scientists).
PR
Main