-
-
Notifications
You must be signed in to change notification settings - Fork 84
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TOML support (tomlkit) is very slow #184
Comments
Thanks for letting me know. I'm beginning to sour on TOML as a file format. I think it's not a great file format. I'm curious if your use case involves writing toml or just extracting values from it? I wonder if I should just discontinue support for writing TOML ( |
No, just extracting the values — actually, the example I gave above is my use-case.
In fact, I very much share your opinion :-) Unfortunately, the Rust ecosystem uses it quite widely (not to say of Python's very own PEP 517 and descendants), and I need to integrate certain processes with it, so here we are.
Yes, but it should be noted that this module is 3.11+. So there likely has to be some sort of a fallback at least for the time being. |
I addressed this by using tomllib when available. |
Parsing a ~700 KiB TOML file with
tomlq
takes 20s on my ~4GHz laptop:Perhaps consider a different toml library if style-preserving features (i. e. roundtrip output) are not required?
The text was updated successfully, but these errors were encountered: