Fix CST parameterization method by Zcaic · Pull Request #2124 · su2code/SU2

Conversation

@Zcaic

Related Work

  1. I add some code about CST to CSurfaceMovement.cpp and it can work well.

  2. I add a test case named "cst" in TestCases/deformation. It contains test case for CST mesh deformation.

PR Checklist

Put an X by all that apply. You can fill this out after submitting the PR. If you have any questions, don't hesitate to ask! We want to help. These are a guide for you to know what the reviewers will be looking for in your contribution.

  • I am submitting my contribution to the develop branch.
  • My contribution generates no new compiler warnings (try with --warnlevel=3 when using meson).
  • My contribution is commented and consistent with SU2 style (https://su2code.github.io/docs_v7/Style-Guide/).
  • I used the pre-commit hook to prevent dirty commits and used pre-commit run --all to format old commits.
  • I have added a test case that demonstrates my contribution, if necessary.
  • I have updated appropriate documentation (Tutorials, Docs Page, config_template.cpp), if necessary.

@Zcaic Zcaic mentioned this pull request

Sep 1, 2023

@pcarruscag pcarruscag changed the title add cst to CSurfaceMovement.cpp Fix CST parameterization method

Sep 1, 2023

pcarruscag

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, add yourself to AUTHORS.md please, since you added it as a testcase, we can try to make a regression test.
Can you check if your config works with the mesh I'm suggesting below (it's in the TestCases repo)? Then you need to change parallel_regression.py (look for /deformation/ to find the other tests).

github-advanced-security[bot]

@Zcaic

@pcarruscag ,
I have modified the cfg file and it passed the test with mpirun -n 2 SU2_DEF naca0012.cfg.
I also modified AUTHORS.md and parallel_regression.py

pcarruscag

pcarruscag

@Zcaic

Sorry, some was caused by my carelessness,. I've modified it.

pcarruscag

pcarruscag

pcarruscag

Labels

2 participants

@Zcaic @pcarruscag