Sticking to the immediate issues and ignoring the other meta issues...
I think what you want could possibly be used to simplify the test
harness if we push things down to the petscdiff level. If we
have petscdiff detect the diff then it will automatically apply
the patches. This would eliminate the "alt" files from the test
harness level.
Of course, petscdiff is in bash and we've talked about replacing
it with a python version. Matt has said he has a preliminary version
and I'd appreciate being able to use this as a starting point.
Scott
On 5/2/19 3:59 PM, Smith, Barry F. wrote:
Scott and PETSc folks,
Using alt files for testing is painful. Whenever you add, for example, a
new variable to be output in a viewer it changes the output files and you need
to regenerate the alt files for all the test configurations. Even though the
run behavior of the code hasn't changed.
I'm looking for suggestions on how to handle this kind of alternative
output in a nicer way (alternative output usually comes from different
iterations counts due to different precision and often even different
compilers).
I idea I was thinking of was instead of having "alt" files we have "patch"
files that continue just the patch to the original output file instead of a complete copy. Thus in
some situations the patch file would still apply even if the original output file changed thus
requiring much less manual work in updating alt files. Essentially the test harness would test
against the output file, if that fails it would apply the first patch and compare again, try the
second patch etc.
Scott,
What do you think? Should be an easy addition to the current model (no
need to even remove the alt testing)? Would it also be possible to add a PATCH
option to the test rule where it automatically added the new patch file?
Perhaps all the patches for a test case could all be stored in the same file
also so we don't need to manage patch_1.out patch_2.out etc? Each new patch
would just get added to the file?
Thoughts?
Barry
--
Tech-X Corporation kru...@txcorp.com
5621 Arapahoe Ave, Suite A Phone: (720) 974-1841
Boulder, CO 80303 Fax: (303) 448-7756