[ET Trac] [Einstein Toolkit] #945: add higher order restriction parameter to cell-centerd Carpet

Einstein Toolkit trac-noreply at einsteintoolkit.org
Mon Jun 4 12:26:22 CDT 2012


#945: add higher order restriction parameter to cell-centerd Carpet
------------------------------------+---------------------------------------
  Reporter:  rhaas                  |       Owner:           
      Type:  enhancement            |      Status:  reopened 
  Priority:  minor                  |   Milestone:           
 Component:  EinsteinToolkit thorn  |     Version:           
Resolution:                         |    Keywords:  CarpetLib
------------------------------------+---------------------------------------
Changes (by eschnett):

  * status:  closed => reopened
  * resolution:  fixed =>


Comment:

 Actually, let's keep the ticket open, we should look at the code again and
 clean it up:

 - The parameter "use_cc_o3" should instead be
 "use_higher_order_restriction" or similar. The actual restriction order
 should then be the same as the prolongation order (up to what is
 available).
 - The code in dh.cc needs to be cleaned up. This concerns the way in which
 the source domain for the operator is enlarged to ensure that the
 restriction stencil fits. I am not convinced that the current code is
 correct in all cases.
 - The implementation of the actual operator could be modernised (although
 this is not necessary).
 - The documentation of the stencil that is given above in this ticket
 should go into the source code.

 There is a remark regarding synchronisation. However, there is no code to
 synchronise. Is Carpet's current behaviour regarding synchronisation
 already correct?

-- 
Ticket URL: <https://trac.einsteintoolkit.org/ticket/945#comment:3>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit


More information about the Trac mailing list