Skip to content

update validation settings in CI test#590

Merged
SeverinDiederichs merged 1 commit intoapt-sim:masterfrom
SeverinDiederichs:update_validation_settings
Apr 23, 2026
Merged

update validation settings in CI test#590
SeverinDiederichs merged 1 commit intoapt-sim:masterfrom
SeverinDiederichs:update_validation_settings

Conversation

@SeverinDiederichs
Copy link
Copy Markdown
Collaborator

As noted in #589, the current validation tests can fail by just changing the random seed, as the fluctuations can lead to an error larger than 1%:

Seed, error:

1234567: pass, 0.407735%
1234568: fail, 1.038702%
1234569: pass, 0.561926%
1234570: pass, 0.556683%
1234571: pass, 0.509937%
1234572: pass, 0.500730%
1234573: pass, 0.944030%
1234574: pass, 0.642265%
1234575: pass, 0.410258%
1234576: fail, 1.347790%

Therefore, this PR increases the number of events from 400 to 600 to increase the statistics and reduce the error more safely below 1%.

To accelerate the CI, the GPU settings on the self-hosted runner are adjusted, such that the GPU is optimally used.

@SeverinDiederichs SeverinDiederichs added the CI Category: related to the continuous integration label Apr 23, 2026
@SeverinDiederichs
Copy link
Copy Markdown
Collaborator Author

/run-test

@phsft-bot
Copy link
Copy Markdown

Can one of the admins verify this patch?

@SeverinDiederichs SeverinDiederichs merged commit 245369f into apt-sim:master Apr 23, 2026
6 checks passed
@SeverinDiederichs SeverinDiederichs deleted the update_validation_settings branch April 23, 2026 19:35
SeverinDiederichs added a commit that referenced this pull request Apr 23, 2026
The hit slot settings in #590 were slightly too high, they failed in the
WDT split test. Reducing it to make it pass
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Category: related to the continuous integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants