Rename and relabel estimator tests as deterministic#5735
Rename and relabel estimator tests as deterministic#5735ye-luo merged 7 commits intoQMCPACK:developfrom
Conversation
|
Removed the acforce estimators since these were wildly different. Potentially special handling needed for mixed precision? |
|
Test this please |
|
|
The few failures previously observed are now gone (?) |
|
@ye-luo : I have not been able to reproduce the previously seen failures either when running the container used for CI or on my own systems (x86,arm). However, checking cdash shows some occasional failures ( https://cdash.qmcpack.org/testSummary.php?project=1&name=estimator-latdev-free-1-16-check&date=2026-01-15 ) in the nightlies. Average ~1/day. The test error output is meaningless out of context, so I will investigate. Probably need to remake these tests completely, making them shorter in particular. This update to include the tests in the CI will therefore wait. |
Pull request was converted to draft
|
@prckent let us move the unstable ones out of determinsitic tests and merge this PR. In the meantime, open an issue to track this tests as we troubleshoot. |
|
Test this please |
|
@ye-luo Updated and issue created. |
Proposed changes
Update the tests/estimator/* tests.
Multiple local runs show these tests to be deterministic. Wider range of build configurations may find we need to revise error bars etc.
What type(s) of changes does this code introduce?
Does this introduce a breaking change?
What systems has this change been tested on?
mac, container, cpu full precision only
Checklist