Fix #2203: Improve Spanwise Section Error Handling in Turbomachinery Simulations#2459
Fix #2203: Improve Spanwise Section Error Handling in Turbomachinery Simulations#2459aryan0931 wants to merge 1 commit intosu2code:developfrom
Conversation
|
This doesn't fix issue #2203 in that the issue of incorrectly determining calculating the spanwise boundary happens because of a bug in SU2, the mesh that is used in this issue is probably fine. I have also encountered this issue and checked the CGNS mesh and it is not malformed. All this PR does is add an additional error catch, please update the description to remove the fix. |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| max = 0; | ||
| SU2_MPI::Allreduce(&MyMin, &min, 1, MPI_DOUBLE, MPI_MIN, SU2_MPI::GetComm()); | ||
| SU2_MPI::Allreduce(&MyMax, &max, 1, MPI_DOUBLE, MPI_MAX, SU2_MPI::GetComm()); | ||
| if (nSpan < 2) { |
Check warning
Code scanning / CodeQL
Comparison result is always the same Warning
| ". Found " + to_string(nSpan) + " nodes along the periodic boundary edge from hub to shroud. " + | ||
| "At least 2 nodes are required. Check your mesh for correct hub-to-shroud edge definition.", | ||
| CURRENT_FUNCTION); | ||
| } |
There was a problem hiding this comment.
This is in the equispaced section of the if statement, it should go with the other error message and run if the automatic section fails to calculate any value of nSpanWiseSections in three dimensions, i.e.
(nSpanWiseSections[INFLOW - 1] == 0 || nSpanWiseSections[OUTFLOW -1] == 0)
| buf, | ||
| CURRENT_FUNCTION); | ||
| string("Turbomachinery simulation requires equal spanwise sections at INFLOW and OUTFLOW. ") + buf + | ||
| "\nThis is likely a mesh issue. Ensure that the periodic boundaries from hub to shroud are consistently defined in the mesh.", |
There was a problem hiding this comment.
This is not likely a mesh issue, it is a mesh issue!
|
Thank you sir for review I will do the respective changes as soon as possible. |
|
Are you going to finish this work @aryan0931? |
3 participants
Proposed Changes
Fixes: #2203
This PR aims to:
This PR improves error messages for spanwise section calculations in turbomachinery simulations, addressing user experience
Related Work
Fixes #2203 : Wrong spanwise sections calculation in Turbomachinery simulation.
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.
pre-commit run --allto format old commits.