[ET Trac] [Einstein Toolkit] #1384: simplify using Refluxing with MoL

Einstein Toolkit trac-noreply at einsteintoolkit.org
Tue Jun 18 22:40:42 CDT 2013


#1384: simplify using Refluxing with MoL
--------------------------+-------------------------------------------------
  Reporter:  rhaas        |       Owner:                     
      Type:  enhancement  |      Status:  new                
  Priority:  optional     |   Milestone:                     
 Component:  Other        |     Version:  development version
Resolution:               |    Keywords:  Refluxing          
--------------------------+-------------------------------------------------

Comment (by rhaas):

 A patch, hmwell, yes, maybe, actually this ticket was more or less
 intended as a reminder for me to get around and provide just such a patch
 (I had assumed that I could use the "enhancement" category for this kind
 of thing).

 I agree that list variables in a string is unusual for MoL. See below for
 a possibly different approach.

 I am not really familiar with what Refluxing currently does, reading
 param.ccl though it seems that the user has to give the full list of
 variables in refluxing_variables and then tell Refluxing which ones to
 register with MoL based in the location in the string, ie. "register only
 the first nvars_evolved" variables with MoL. I am not sure if this is a
 very robust interface since it requires several parameters to be
 consistent with each other.

 A different method would be a version where Refluxing takes the full list
 of variables, then queries MoL whether each variable is registered with
 MoL as an evolved variable (via a call to QueryRHS or whatever the aliased
 function is actually called) and only registers those fluxes with MoL that
 have RHS registered already. This would do away with the two separate
 parameters nvars_evolved and nvars altogether, which I would like very
 much.

 Please note that this ticket is in not way intended as a "can someone
 please write this for me" ticket, but really only as a reminder to myself
 to write something like this or to gather opinions if this is a good
 interface.

 No patch yet that implements this either.

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


More information about the Trac mailing list