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

process-reports: taint subtest timeouts by suspicion #85

Merged
merged 5 commits into from
Apr 15, 2024

Conversation

ErichDonGubler
Copy link
Owner

Currently, if one is running a WebGPU CTS test such that there's not
a deterministic place it will time out (like the tests with thousands of
tiny subtest cases that simply don't fit in the time budget wptrunner
gives them), it can take dozens to hundreds of runs to empirically
observe every single potential point of concretely timing out. This is
a problem when the goal is to converge expected outcomes in as few runs
as possible!

Resolve this as follows: If over the course of processing we encounter
any reconciled subtest outcomes that contain a TIMEOUT or NOTRUN
value, ensure both TIMEOUT and NOTRUN are present in the
reconciled subtest outcomes.

Currently, if one is running a WebGPU CTS test such that there's not
a deterministic place it will time out (like the tests with thousands of
tiny subtest cases that simply don't fit in the time budget `wptrunner`
gives them), it can take dozens to hundreds of runs to empirically
observe every single potential point of concretely timing out. This is
a problem when the goal is to converge expected outcomes in as few runs
as possible!

Resolve this as follows: If over the course of processing we encounter
any reconciled subtest outcomes that contain a `TIMEOUT` or `NOTRUN`
value, ensure _both_ `TIMEOUT` and `NOTRUN` are present in the
reconciled subtest outcomes.
@ErichDonGubler ErichDonGubler self-assigned this Apr 15, 2024
@ErichDonGubler ErichDonGubler added the enhancement New feature or request label Apr 15, 2024
@ErichDonGubler ErichDonGubler merged commit 0a909af into main Apr 15, 2024
18 checks passed
@ErichDonGubler ErichDonGubler deleted the taint-reported-timeouts-by-suspicion branch April 15, 2024 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant