-
Notifications
You must be signed in to change notification settings - Fork 845
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add additional checks with warning/error messages for turbomachinery cases. #2378
Comments
I think a refactor of the turbovertex structure would be very beneficial in the long run and will address points 1 & 2, maybe this would be better from a user perspective rather than error messgaes. If we're going to add something that determines the relative orientation of the boundaries, we might as well make it so the geometric preprocessing can use this information to build the turbovertex structure for machines with arbitary inflow/outflow orientation. Currently mixed-flow machines are not possible in develop, we could allow these types of machine with this. We will also probably slim down the code quite significantly and bring large section up to date with C++ 11 standards. |
Hi @bigfooted and @joshkellyjak , I would like to work on this issue as part of my contribution to SU2. I plan to implement the required checks and add appropriate warning/error messages for turbomachinery cases. Thank you! |
Hi Vaishnavi, great to hear that you would like to work on improving the turbomachinery workflow. Are you currently working on a turbomachinery case that you can use as a testcase? |
Good to hear! Could you join a developer meeting to discuss this further? @EvertBunschoten and I also discussed this yesterday, I think some coordination is in order |
Yes, I have been working on simple turbomachinery case. |
Yeah sure, please let me know the details. |
The next meeting is EU time, the details can be found in the slack channel. If you need to find the link to join the channel you can find it at the top of https://su2code.github.io/ |
Is your feature request related to a problem? Please describe.
In a discussion on cfd-online we concluded that some warning/error messages will be helpful to guide users in setting up turbomachinery cases.
https://www.cfd-online.com/Forums/su2/258309-su2-compressor-turbomachinery-case.html
Conclusion: Warning/error messages for the 3 points below will be very useful:
The text was updated successfully, but these errors were encountered: