Skip to content

Updated CFS test with current values to correct Master MR issue

Spencer Sherwin requested to merge fix/CFS_Test into master

Issue/feature addressed

After merging two requests to master there was a conflict arising from the overlapping merges in one test.

Proposed solution

Modified the test values and tolerances to allow test to pass

Implementation

As above

Tests

Test was CompressibleFlowSolver_ShockTube_2D_mixedMesh_AV_Phys_Dil_IP_Impl

Notes

Checklist

  • Functions and classes, or changes to them, are documented.
  • User guide/documentation is updated.
  • Changelog is updated.
  • Suitable tests added for new functionality.
  • Contributed code is correctly formatted. (See the contributing guidelines).
  • License added to any new files.
  • No extraneous files have been added (e.g. compiler output or test data files).

Warning

On the 19.07 the code formatting (code style) was standardised using clang-format, over the whole Nektar++ code. This means changes in your branch will conflict with formatting changes on the master branch. To resolve these conflicts , see #295 (closed)

Edited by Spencer Sherwin

Merge request reports