Skip to content
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

Update benchmarking #395

Merged
merged 12 commits into from
Mar 23, 2020
Merged

Update benchmarking #395

merged 12 commits into from
Mar 23, 2020

Conversation

hugovk
Copy link
Member

@hugovk hugovk commented Mar 23, 2020

Changes proposed in this pull request:

  • Remove little-used yajl (882 downloads in Feb), https://github.com/rtyler/py-yajl has been archived
  • Add orjson and nujson
  • Add handling for orjson's "recursion limit exceeded" for complex objects, just skips
  • Comma separate thousands in the output for readability, and no need for 2 decimal places, the numbers are from the tens to the tens of thousands
  • Update README with benchmarked data, using last ujson release, on Linux on GHA
  • Benchmark dev version on pushes to master. Not other branches or PRs, because it can take up to 15 mins to run
  • Could try on other Python versions and other operating systems too at some point

@hugovk hugovk added the changelog: Changed For changes in existing functionality label Mar 23, 2020
@hugovk hugovk merged commit 22875bf into ultrajson:master Mar 23, 2020
@hugovk hugovk deleted the benchmark-on-ci branch March 23, 2020 18:26
@hugovk hugovk mentioned this pull request Dec 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: Changed For changes in existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant