Skip to content

Commit

Permalink
Report differences on sheath restart test
Browse files Browse the repository at this point in the history
The sheath restart test is occasionally failing. As detailed in !211 this is probably because the test requires bit reproducibility however it would be good to confirm this. This is tricky as the bug is not reliably reproducible. This MR therefore adds the `-r` flag to the `h5diff` command. This reports the differences which will allow us to have more information about this error the next time it appears.

See merge request gysela-developpers/gyselalibxx!452

--------------------------------------------
  • Loading branch information
EmilyBourne committed Apr 8, 2024
1 parent 8173fc1 commit 16b15c4
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions tests/geometryXVx/sheath/test_sheath_restart.sh
Original file line number Diff line number Diff line change
Expand Up @@ -45,21 +45,21 @@ cp "${RSTDIR}/sheath.yaml" sheath_restart.yaml
sed -i 's/^ nbiter: .*/ nbiter: 2/' sheath_restart.yaml
sed -i 's/^ time_diag: .*/ time_diag: 0.5/' sheath_restart.yaml

h5ls -d ${PWD}/VOICEXX_00006.h5/time_saved ${RSTDIR}/VOICEXX_00005.h5/time_saved
command="h5diff ${PWD}/VOICEXX_00005.h5 ${RSTDIR}/VOICEXX_00005.h5 time_saved"
h5ls -d ${PWD}/VOICEXX_00005.h5/time_saved ${RSTDIR}/VOICEXX_00005.h5/time_saved
command="h5diff -r ${PWD}/VOICEXX_00005.h5 ${RSTDIR}/VOICEXX_00005.h5 time_saved"
eval $command
if [ $? -ne 0 ]; then
exit 1
fi

command="h5diff ${PWD}/VOICEXX_00005.h5 ${RSTDIR}/VOICEXX_00005.h5 electrostatic_potential"
command="h5diff -r ${PWD}/VOICEXX_00005.h5 ${RSTDIR}/VOICEXX_00005.h5 electrostatic_potential"
eval $command
if [ $? -ne 0 ]; then
exit 1
fi

command="h5diff ${PWD}/VOICEXX_00005.h5 ${RSTDIR}/VOICEXX_00005.h5 fdistribu"
command="h5diff -r ${PWD}/VOICEXX_00005.h5 ${RSTDIR}/VOICEXX_00005.h5 fdistribu"
eval $command
if [ $? -ne 0 ]; then
exit 1
fi
fi

0 comments on commit 16b15c4

Please sign in to comment.