(BREAKING) refactor: rename dropWorst option to warmupRuns

[mdjastrzebski]

Summary

Based on the developer feedback the dropWorst parameter was not well named nor understood. This PR changes the name to warmupRuns and slightly tweak the logic to discard first runs and not the highest runes (these were usually the same runs).

Test plan

Relevant tests have been tweaked.

[changeset-bot]

🦋 Changeset detected

Latest commit: ace21d9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages

Name	Type
@callstack/reassure-measure	Minor
reassure	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[github-actions]

Performance Comparison Report

• Current: 72bb044 - 2023-05-09 10:11:00Z
• Baseline: main (baf90de) - 2023-05-09 10:10:20Z

Significant Changes To Render Duration

Name	Render Duration	Render Count
Async Component	252.3 ms → 243.0 ms (-9.3 ms, -3.7%)	7 → 7

Show details

Name	Render Duration	Render Count
Async Component	Baseline Mean: 252.3 ms Stdev: 6.8 ms (2.7%) Runs: 264 258 256 255 254 252 251 249 243 241 Current Mean: 243.0 ms Stdev: 9.4 ms (3.9%) Runs: 259 252 249 246 244 242 240 240 230 228	Baseline Mean: 7 Stdev: 0 (0.0%) Runs: 7 7 7 7 7 7 7 7 7 7 Current Mean: 7 Stdev: 0 (0.0%) Runs: 7 7 7 7 7 7 7 7 7 7

Meaningless Changes To Render Duration

Show entries

Name	Render Duration	Render Count
Other Component 20	151.5 ms → 151.0 ms (-0.5 ms, ±0.0%)	4 → 4
Other Component 10	149.4 ms → 146.9 ms (-2.5 ms, -1.7%)	4 → 4
Other Component 10 legacy scenario	152.7 ms → 150.1 ms (-2.6 ms, -1.7%)	4 → 4

Show details

Name	Render Duration	Render Count
Other Component 20	Baseline Mean: 151.5 ms Stdev: 5.2 ms (3.5%) Runs: 159 159 157 157 156 156 154 154 154 152 152 151 150 149 147 147 146 145 144 141 Current Mean: 151.0 ms Stdev: 4.9 ms (3.3%) Runs: 164 160 156 154 154 153 152 151 151 150 150 150 150 148 148 147 147 146 145 144	Baseline Mean: 4 Stdev: 0 (0.0%) Runs: 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 Current Mean: 4 Stdev: 0 (0.0%) Runs: 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4 4
Other Component 10	Baseline Mean: 149.4 ms Stdev: 6.4 ms (4.3%) Runs: 155 155 155 155 152 150 147 145 144 136 Current Mean: 146.9 ms Stdev: 6.9 ms (4.7%) Runs: 159 153 153 151 146 145 144 142 138 138	Baseline Mean: 4 Stdev: 0 (0.0%) Runs: 4 4 4 4 4 4 4 4 4 4 Current Mean: 4 Stdev: 0 (0.0%) Runs: 4 4 4 4 4 4 4 4 4 4
Other Component 10 legacy scenario	Baseline Mean: 152.7 ms Stdev: 7.5 ms (4.9%) Runs: 161 160 159 158 154 153 152 149 142 139 Current Mean: 150.1 ms Stdev: 7.2 ms (4.8%) Runs: 164 157 156 152 150 148 145 144 143 142	Baseline Mean: 4 Stdev: 0 (0.0%) Runs: 4 4 4 4 4 4 4 4 4 4 Current Mean: 4 Stdev: 0 (0.0%) Runs: 4 4 4 4 4 4 4 4 4 4

Changes To Render Count

There are no entries

Added Scenarios

There are no entries

Removed Scenarios

There are no entries

Generated by 🚫 dangerJS against ace21d9

[thymikee]

I think that's better