pfhub icon indicating copy to clipboard operation
pfhub copied to clipboard

Updated the BM7 spec based on feedback from the workshop

Open stvdwtt opened this issue 6 years ago • 4 comments

Primary changes under consideration:

  1. Remove part C (or perhaps change it to have the interfacial width of part A)
  2. Look into reducing the total simulated time, parts B and C might be too computationally intensive
  3. Change the description of how the temporal OOA is calculated, requesting that the approximate spatial error is subtracted off (and the same for the spatial OOA?)
  4. Fix the SymPy code
  5. Explain the "L" that shows up in the auto-generated C code
  6. Fix the alpha->a substitution that appears in the auto-generated C code
  7. Modify the early narrative to include the source term in the original equation and then solve for it (likely more intuitive than the current set up)
  8. Change the requested tasks/uploads to bullets from blocks of text
  9. Emphasize that convergence tests are not required for part B (and C if extant)

stvdwtt avatar Mar 01 '18 13:03 stvdwtt