#1417: Fixed refinement levels break AMR ----------------------------------+----------------------------------------- Reporter: coryell@… | Owner: eschnett Type: defect | Status: closed Priority: major | Milestone: Component: Carpet | Version: development version Resolution: fixed | Keywords: CarpetRegrid2 ----------------------------------+-----------------------------------------
Comment (by rhaas):
Works for me, both codes seem to be functionally identical. The modified code still regrids and passes all tests. I actually had something similar to what is now implemented as a first try but thought that proposing {{{ do_recompose = false; if (adaptive_refinement) do_recompose = true; else for (int n = 0; n < num_centres; ++ n) { ...69 lines of code... } }}} likely would have had me scolded for not indenting the loop :-). The current code is obviously closer in style to what is used in the file otherwise.
Thank you for reviewing and committing the patch.