Skip to content
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

[WIP] Removing turbomachinery legacy output #2001

Closed
wants to merge 77 commits into from
Closed

[WIP] Removing turbomachinery legacy output #2001

wants to merge 77 commits into from

Conversation

joshkellyjak
Copy link
Contributor

Proposed Changes

Transferring turbomachinery outputs from legacy format to current format

Related Work

Resolve any issues (bug fix or feature request), note any related PRs, or mention interactions with the work of others, if any.

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 have added a test case that demonstrates my contribution, if necessary.
  • I have updated appropriate documentation (Tutorials, Docs Page, config_template.cpp), if necessary.

salvovitale and others added 30 commits February 8, 2020 23:33
Conflicts:
	SU2_CFD/include/solvers/CEulerSolver.hpp
	SU2_CFD/src/integration_time.cpp
	SU2_CFD/src/iteration_structure.cpp
	SU2_CFD/src/meson.build
	SU2_CFD/src/solvers/CEulerSolver.cpp
	SU2_CFD/src/solvers/CNSSolver.cpp
…ery boundary value table added for priting, printing frequency is still hard coded, to be fixed in future
…ing Plane under process. Turbine and Compressor calcualtion if statement added
…e performance is not computer for all blades. This needs to be identified and implemented
…ixed and turbo span calculation to be fixed.
…. Mutizone is still crashing... investigation in progress
@bigfooted bigfooted changed the title Removing turbomachinery legacy output [WIP] Removing turbomachinery legacy output Apr 4, 2023
@pcarruscag
Copy link
Member

Maybe we can chat in the developer meeting today at 4pm CET (in 2.5h) https://meet.jit.si/SU2_DevMeeting it looks like this is becoming harder than it needs to 😅

@joshkellyjak
Copy link
Contributor Author

Sounds good, see you then

@pcarruscag
Copy link
Member

see #2003

@joshkellyjak joshkellyjak closed this by deleting the head repository Apr 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants