Neutron star gallery example produces output different from what is shown on website in ET_Tesla

Comments (17)

1. 

   Roland Haas reporter

   • changed milestone to ET_2018_02
   • removed comment

   • 2018-02-05T15:38:06+00:00
2. 

   Roland Haas reporter

   • removed comment

   I will try and see if I can find out if there is any ET release that reproduces the plot on the website.

   Note that until #2117 is fixed one cannot run the parfile with more than one MPI rank and the proposed release code.

   • 2018-02-05T15:39:22+00:00
3. 

   Ian Hinder

   • marked as
   • removed comment

   • 2018-02-06T03:22:55+00:00
4. 

   Ian Hinder

   • changed status to open
   • removed comment

   • 2018-02-06T03:23:22+00:00
5. 

   Ian Hinder

   • removed comment

   • 2018-02-06T03:42:19+00:00
6. 

   Ian Hinder

   • removed comment

   • 2018-02-06T03:42:57+00:00
7. 

   Roland Haas reporter

   • removed comment

   • 2018-02-06T07:21:00+00:00
8. 

   Roland Haas reporter

   • removed comment

   • 2018-02-06T07:47:50+00:00
9. 

   Roland Haas reporter

   • removed comment

   The difference shows up between the Herschel (good) and Hilbert (bad) releases:

   || Herschel || 2014-11 || [[Image(rho-Herschel.png, 50%)]] || || Hilbert || 2015-05 || [[Image(rho-Hilbert.png, 50%)]] ||

   which happens to be when the 'default' code in GRHydro switched from F90 (Herschel) to CXX (Hilbert). However flipping the use_cxx_code parameter does not affect the output so that it does not seem to be a difference between CXX and F90 code but something else that changed in between the releases.

   • 2018-02-06T09:19:55+00:00
10. 

    Roland Haas reporter

    • removed comment

    I ran git bisect which eventually reports:

    commit cef0ee829c329d5edbfb288a94c8379b2fa32d62
    Author: Frank Loeffler <knarf@cct.lsu.edu>
    Date:   Mon Nov 24 09:25:42 2014 -0600
    
        GRHydro: remove dependency on ADMMacros::spatial_order
    
        Introduce GRHydro::sources_spatial_order in favor of
        ADMMacros::spatial_order. Only the location (and name) of the parameter
        changed.
    

    so I will try what happens if I use the new parameter GRHydro::sources_spatial_order.

    • 2018-02-06T09:52:55+00:00
11. 

    Roland Haas reporter

    • changed status to resolved
    • removed comment

    Yup, that was it, adding "GRHydro::sources_spatial_order = 4" restores the old plot:

    [[Image(rho-source_spatial_order4.png, 50%)]]

    I have updated the parfile in the gallery.

    Lesson for the future:

    Though shalt not silently change default behaviour, for backwards compatibility is a fickle god and will be punishing the maintainers for the sin of the developers to the third and fourth generation of those who hate it.

    or so.

    • 2018-02-06T11:02:08+00:00
12. 

    Ian Hinder

    • removed comment

    We should probably tell people about this on the mailing list, in case they have some of their own parfiles which need fixing. This will be good even if #2120 is implemented.

    • 2018-02-07T03:44:15+00:00
13. 

    Frank Löffler

    • removed comment

    See also #1642 for reference.

    • 2018-02-07T15:20:01+00:00
14. 

    Roland Haas reporter

    • changed status to closed
    • edited description

    • 2019-02-21T19:50:43+00:00
15. 

    Roland Haas reporter

    • assigned issue to
      
      Roland Haas
    • edited description

    • 2019-04-28T17:49:37+00:00
16. 

    Roland Haas reporter

    • edited description

    • 2019-04-28T17:51:36+00:00
17. 

    Roland Haas reporter

    • edited description

    • 2019-04-28T17:54:13+00:00
18. Log in to comment