Run the unsteady NACA0012 tutorial for 20 timesteps by pcarruscag · Pull Request #1616 · su2code/SU2
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 971
Merged
Run the unsteady NACA0012 tutorial for 20 timesteps#1616
pcarruscag merged 4 commits intodevelopfrom
Run the unsteady NACA0012 tutorial for 20 timesteps#1616
pcarruscag merged 4 commits intodevelopfrom
Conversation
Copy link
Member
Starts to address #1604
The adjoint will be more work.
Depends on su2code/Tutorials#22
pcarruscag
added
the
changelog:chore
label
pr-triage
bot
added
the
PR: unreviewed
label
Copy link
lgtm-com
bot
commented
Apr 23, 2022
lgtm-com bot commented
Apr 23, 2022This pull request fixes 3 alerts when merging cd5c18f into f4f6298 - view on LGTM.com
fixed alerts:
- 3 for Comparison of narrow type with wide type in loop condition
pcarruscag commented Apr 23, 2022
TestCases/tutorials.py
Outdated
Show resolved
Hide resolved
TestCases/tutorials.py Outdated Show resolved Hide resolved
Copy link
lgtm-com
bot
commented
Apr 23, 2022
lgtm-com bot commented
Apr 23, 2022This pull request fixes 3 alerts when merging 5674c23 into f4f6298 - view on LGTM.com
fixed alerts:
- 3 for Comparison of narrow type with wide type in loop condition
TobiKattmann approved these changes Apr 23, 2022
Copy link
Contributor
TobiKattmann
left a comment
•
edited
Loading
TobiKattmann
left a comment
•
edited
Loading
edited
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for making this Tutorial work nicely again and the additional cleanup on the output stuff 💐 Everything lgtm so feel free to merge this and the Tutorial PR from my side
pr-triage
bot
added
PR: reviewed-approved
and removed
PR: unreviewed
labels
pcarruscag commented May 10, 2022
.github/workflows/regression.yml
Outdated
Show resolved
Hide resolved
.github/workflows/regression.yml Outdated Show resolved Hide resolved
pr-triage
bot
added
PR: unreviewed
and removed
PR: reviewed-approved
labels
pcarruscag
merged commit
fa910a0
into
develop
pcarruscag
deleted the
improve_unsteady_tutorials
branch
pr-triage
bot
added
PR: merged
and removed
PR: unreviewed
labels
Copy link
lgtm-com
bot
commented
May 10, 2022
lgtm-com bot commented
May 10, 2022This pull request fixes 3 alerts when merging 3fa0e2a into 6acc62f - view on LGTM.com
fixed alerts:
- 3 for Comparison of narrow type with wide type in loop condition
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment