PITTNullCode updates

Comments (7)

1. 

   Roland Haas reporter

   • changed status to open
   • removed comment

   • 2012-07-28T03:34:44+00:00
2. 

   anonymous

   • removed comment

   NullNews.patch seems to be a patch for NullSHRExtract. It also conflicts with NullSHRExtract.patch

   • 2012-07-30T16:12:18+00:00
3. 

   Roland Haas reporter

   • removed comment

   Are you sure you are looking at the newest patches? There were wrong patches in before but they were corrected today at 9AM. The NullNews one should start as:

   Index: param.ccl

   --- param.ccl (revision 9) +++ param.ccl (working copy) @@ -22,19 +22,19 @@

   private:

   -BOOLEAN first_order_scheme "should angular derviatives be reduced to first order?" +BOOLEAN first_order_scheme "should angular derviatives be reduced to first order?" STEERABLE=ALWAYS { } "yes"

   They are all to be applied in their respective thorn directory it NullNews.patch from within PITTNullCode/NullNews and not from within PITTNullCode.

   • 2012-07-30T16:26:47+00:00
4. 

   anonymous

   • removed comment

   It's actually NullVars.patch that seems to be the problem.

   • 2012-07-31T13:58:32+00:00
5. 

   anonymous

   • removed comment

   Please commit these changes. They don't affect the testsuites and Bela found them to be essential for the correctness of the code.

   The start-up related change could be logged as

   Improve the start-up algorithm of the characteristic marching scheme from the inner boundary towards scri+.

   All other changes are related to either improving IO (will correctly truncate/create diagnostic files) or allowing for flexible tuning of IO between checkpoints/restarts (by changing the parameters into steerable ones).

   • 2012-07-31T14:01:11+00:00
6. 

   Roland Haas reporter

   • changed status to resolved
   • removed comment

   Thank you for the review. Yes NullVars was indeed faulty and had not been fixed. My apologies. I have applied all patches as revisions 9 of NullDecomp, 10 of NullEvolve, 10 of NullNews and 10 of NullSHRExtract,

   • 2012-07-31T16:06:53+00:00
7. 

   Roland Haas reporter

   • edited description
   • changed status to closed

   • 2019-02-21T20:13:45+00:00
8. Log in to comment