Implement a new LoopPropertiesNoAdjacentAnnotation compile step
#1076
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Benchmark (linux/gcc)
| Benchmark suite | Current: 7b52afe | Previous: 7789be6 | Ratio |
|---|---|---|---|
JSONSchema_Validate_Draft4_Meta_1_No_Callback |
6163.653173572586 ns/iter |
7684.8167530219835 ns/iter |
0.80 |
JSONSchema_Validate_Draft4_Required_Properties |
3491.0172010820656 ns/iter |
3448.3168521620123 ns/iter |
1.01 |
JSONSchema_Validate_Draft4_Optional_Properties_Minimal_Match |
819.8035490285007 ns/iter |
821.8162443909573 ns/iter |
1.00 |
JSONSchema_Validate_Draft4_Items_Schema |
12116.70292462174 ns/iter |
11951.22582302829 ns/iter |
1.01 |
JSONSchema_Compile_Basic |
388072.84193012305 ns/iter |
389227.54550500284 ns/iter |
1.00 |
This comment was automatically generated by workflow using github-action-benchmark.
jviotti
force-pushed
the
loop-properties-adjacent-annotation
branch
3 times, most recently
from
e6180c1
to
5b96ed8
Compare
There was a problem hiding this comment.
Benchmark (macos)
| Benchmark suite | Current: 7b52afe | Previous: 7789be6 | Ratio |
|---|---|---|---|
JSONSchema_Compile_Basic |
197123.42587292523 ns/iter |
206830.07481524645 ns/iter |
0.95 |
JSONSchema_Validate_Draft4_Meta_1_No_Callback |
6099.936029245597 ns/iter |
7392.894571660287 ns/iter |
0.83 |
JSONSchema_Validate_Draft4_Required_Properties |
3450.036263239196 ns/iter |
3654.227698522444 ns/iter |
0.94 |
JSONSchema_Validate_Draft4_Optional_Properties_Minimal_Match |
1206.259106709967 ns/iter |
1306.3695505374967 ns/iter |
0.92 |
JSONSchema_Validate_Draft4_Items_Schema |
9139.998942424638 ns/iter |
10439.41741670299 ns/iter |
0.88 |
This comment was automatically generated by workflow using github-action-benchmark.
There was a problem hiding this comment.
Benchmark (linux/llvm)
| Benchmark suite | Current: 7b52afe | Previous: 7789be6 | Ratio |
|---|---|---|---|
JSONSchema_Compile_Basic |
802035.1051671291 ns/iter |
386739.16954822233 ns/iter |
2.07 |
JSONSchema_Validate_Draft4_Meta_1_No_Callback |
14087.521096401266 ns/iter |
7462.441005761438 ns/iter |
1.89 |
JSONSchema_Validate_Draft4_Required_Properties |
6807.355295068178 ns/iter |
3274.4145738987377 ns/iter |
2.08 |
JSONSchema_Validate_Draft4_Optional_Properties_Minimal_Match |
1738.5921429597681 ns/iter |
777.0137173085205 ns/iter |
2.24 |
JSONSchema_Validate_Draft4_Items_Schema |
24568.013961419583 ns/iter |
10378.975223022591 ns/iter |
2.37 |
This comment was automatically generated by workflow using github-action-benchmark.
Signed-off-by: Juan Cruz Viotti <[email protected]>
jviotti
force-pushed
the
loop-properties-adjacent-annotation
branch
from
5b96ed8
to
7b52afe
Compare
There was a problem hiding this comment.
Benchmark (windows)
| Benchmark suite | Current: 7b52afe | Previous: 7789be6 | Ratio |
|---|---|---|---|
JSONSchema_Compile_Basic |
777076.562499955 ns/iter |
784526.5625001017 ns/iter |
0.99 |
JSONSchema_Validate_Draft4_Meta_1_No_Callback |
13128.667282735927 ns/iter |
15602.646892724584 ns/iter |
0.84 |
JSONSchema_Validate_Draft4_Required_Properties |
5419.186999999965 ns/iter |
5337.157999999818 ns/iter |
1.02 |
JSONSchema_Validate_Draft4_Optional_Properties_Minimal_Match |
2385.761785714229 ns/iter |
2289.679999999805 ns/iter |
1.04 |
JSONSchema_Validate_Draft4_Items_Schema |
26934.9182256277 ns/iter |
25856.357142856723 ns/iter |
1.04 |
This comment was automatically generated by workflow using github-action-benchmark.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Juan Cruz Viotti [email protected]