Skip to content
This repository has been archived by the owner on Jan 12, 2024. It is now read-only.

BayesianPEIsCorrect test fails probabilistically #622

Open
tcNickolas opened this issue Mar 22, 2022 · 1 comment
Open

BayesianPEIsCorrect test fails probabilistically #622

tcNickolas opened this issue Mar 22, 2022 · 1 comment
Labels
Kind-Maintenance Request or suggestion for general code quality improvements

Comments

@tcNickolas
Copy link
Member

Describe the bug
BayesianPEIsCorrect test fails probabilistically.
Similarly to microsoft/QuantumLibraries#444, we should fix the seed used in the test, or run several tries in a loop and say that the test passes if at least one of the runs succeeds.

To Reproduce
The last failure I've seen.

@cgranade cgranade added the Kind-Maintenance Request or suggestion for general code quality improvements label Sep 27, 2022
@cgranade
Copy link
Contributor

At least for fixing the seed, that's unfortunately not currently supported (as per your earlier issue at microsoft/qsharp-runtime#947). In the meantime, adding this to the maintenance backlog to consider your suggestion of looping several times.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Kind-Maintenance Request or suggestion for general code quality improvements
Projects
None yet
Development

No branches or pull requests

2 participants