See how much new version of python-benedict impacts us
See original GitHub issueThe recent release of python-benedict
has monkeypatched JSONEncoder
to always use python-based encoder (instead of C-based encoder.
This monkeypatching happens during import. https://github.com/iterative/dvc/blob/a7f01b65a5cd1654b74e4aa5651063f2b118831d/dvc/utils/collections.py#L85
As we only use it during exp run --set-params
, we need to see how much it affects us. Also how important is the performance for the exp run
? Do we save large directories and files during exp run
?
Note: Simple benchmark shows it causes 2x-4x degradation.
Issue Analytics
- State:
- Created 2 years ago
- Reactions:1
- Comments:8 (3 by maintainers)
Top Results From Across the Web
python-benedict - PyPI
Latest version. Released: Nov 26, 2022. python-benedict is a dict subclass with keylist/keypath support, normalized I/O operations (base64, csv, ini, json, ...
Read more >Python-benedict: Dict Subclass with Keylist/keypath Support
python -benedict is a dict subclass with keylist/keypath support, I/O shortcuts ( base64 , csv , ini , json , pickle , plist...
Read more >Released python-benedict 0.24.0 - Reddit
Hello! I just released an update of python-benedict with .ini files format support. For those who don't know it yet, it's a dict...
Read more >python-benedict - Read the Docs
python -benedict · Overview · Downloads · Search · Builds · Versions. Versions.
Read more >Issues · fabiocaccamo/python-benedict - GitHub
... csv, ini, json, pickle, plist, query-string, toml, xls, xml, yaml), s3 support and many utilities. - Issues · fabiocaccamo/python-benedict.
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
May be worth revisiting https://github.com/iterative/dvc/issues/4883
Discussed with @efiop to get rid of the dependency, we only use it in
--set-params
. We need to see what it’ll take to replace it.