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

Improve serializing by using experimental reverse schema under the hood #89

Merged
merged 25 commits into from
Sep 10, 2024

Conversation

DZakh
Copy link
Owner

@DZakh DZakh commented Aug 31, 2024

No description provided.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Benchmark

Benchmark suite Current: 7f0a9f0 Previous: 83f51d8 Ratio
Parse string 817592668 ops/sec (±0.29%) 815414023 ops/sec (±0.49%) 1.00
Serialize string 820529008 ops/sec (±0.07%) 817783951 ops/sec (±0.11%) 1.00
Advanced object schema factory 480590 ops/sec (±0.37%) 483577 ops/sec (±0.73%) 1.01
Parse advanced object 57537992 ops/sec (±0.33%) 57257765 ops/sec (±0.43%) 1.00
Assert advanced object 173887408 ops/sec (±0.17%) 173214107 ops/sec (±0.13%) 1.00
Create and parse advanced object 94776 ops/sec (±0.73%) 94633 ops/sec (±0.16%) 1.00
Parse advanced strict object 25486040 ops/sec (±0.35%) 25373330 ops/sec (±0.31%) 1.00
Assert advanced strict object 30657194 ops/sec (±0.54%) 30917166 ops/sec (±0.16%) 1.01
Serialize advanced object 56142511 ops/sec (±3.29%) 76333117 ops/sec (±0.06%) 1.36

This comment was automatically generated by workflow using github-action-benchmark.

Copy link

codecov bot commented Aug 31, 2024

Codecov Report

Attention: Patch coverage is 97.86667% with 16 lines in your changes missing coverage. Please review.

Project coverage is 97.34%. Comparing base (95f5a23) to head (39b79fe).
Report is 13 commits behind head on main.

Files with missing lines Patch % Lines
src/S_Core.bs.mjs 98.08% 13 Missing and 1 partial ⚠️
packages/tests/src/utils/U.bs.mjs 88.23% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
- Coverage   97.46%   97.34%   -0.13%     
==========================================
  Files          12       12              
  Lines        5803     5941     +138     
  Branches      793      819      +26     
==========================================
+ Hits         5656     5783     +127     
- Misses        145      155      +10     
- Partials        2        3       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@DZakh DZakh merged commit 9fde7c6 into main Sep 10, 2024
4 checks passed
@DZakh DZakh deleted the reverse-2 branch September 10, 2024 16:43
@DZakh DZakh changed the title Reverse - Second try Improve serializing by using experimental reverse schema under the hood Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant