[Carpet] [Carpet-darcs] CarpetEvolutionMask: fix 2D output for CarpetEvolutionMask_test testsuite

Thomas Radke tradke at aei.mpg.de
Fri Jan 26 13:42:22 CET 2007


Erik Schnetter wrote:
> On Dec 20, 2006, at 04:59:27, Carpet-Repository wrote:
> 
>> The Carpet repository at ~darcs/carpet received a new patch:
>>
>>    CarpetEvolutionMask: fix 2D output for CarpetEvolutionMask_test  
>> testsuite
>>
>> This patch was recorded by Thomas Radke <tradke at aei.mpg.de> at  
>> 2006-12-19 20:58:34.
>>
>> The long comment for this patch reads:
>>
>>  All it took was to change IO::out_fileinfo="none" into  
>> IO::out_fileinfo="axis labels".
> 
> 
> This test case does not pass for me.  The repository contains *.xg  
> files for the norms, which is very strange for CarpetIOScalar output;  
> there should be *.asc files.
> 
> Thomas, can you check this test case again?

I did, and it turns out that I made a mistake there being inconsistent 
in fixing this testsuite (sorry about that).

One problem was that the 1D/2D CarpetIOASCII files contain header 
information which is produced only with IO::out_fileinfo = "axis 
labels". That I fixed with this patch.
However, another problem still remained: the testsuite still contains 
the norms from CactusBase/IOBasic which are not compatible with 
CarpetIOScalar output. When creating the patch I had switched back to 
using thorn IOBasic (in its old version from before 1 Nov 2005 which 
doesn't use the new reduction API yet) in the testsuite parfile. The 
testsuite would now succeed for all the norms but only with 
IO::out_fileinfo still set to "none" as before.

Anyway, I believe the testsuite is still passing in principle. One only 
needs to fix the header information in the 1D/2D ASCII output files and 
recreate the norms in CarpetIOScalar format. Would you agree ? If so I'd 
create another patch.

-- 
Cheers, Thomas.



More information about the developers mailing list