Skip to content

Conversation

@pcarruscag
Copy link
Member

@TobiKattmann
Copy link
Contributor

And the addition of a mesh and restart files is done in su2code/TestCases#101

Copy link
Contributor

@TobiKattmann TobiKattmann left a comment

Choose a reason for hiding this comment

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

Although marked as WIP I guess this is safe to approve ;) and of course here as well thanks for this vandv effort 💐 (of course if you plan to add more I'll take another look 👍 )

I really like your cfg "sections" ... very well made (I usually like the little header with author, date, etc. but I also see why that is not really necessary)

@aeroamit
Copy link
Contributor

@pcarruscag, Nice.....
I saw the results and cross checked with reference results, they seems to be matching. I also observed that you have changed the mesh (adapted+boundary layer continuation for top wall).
Seems like mesh was the culprit earlier?? (though inlet still seems to have minor issue but doesn't matter)
Also see the attached experiment data file (in case you want to use)...
Schulein_2004_expt.dat.gz

Regards
Amit

@pcarruscag pcarruscag changed the title [WIP] Add V&V case, Mach 5 Shock-Wave Boundary-Layer Interaction Add V&V case, Mach 5 Shock-Wave Boundary-Layer Interaction May 31, 2022
Copy link
Member Author

@pcarruscag pcarruscag left a comment

Choose a reason for hiding this comment

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

Thank you @aeroamit, the intersection of supersonic inlet with the bottom wall was causing problems, I extended the inlet region 10mm and used a slip wall for the extension, the small disturbance is also visible in other numerical results in the NPARC website.
I have used the experimental data for the V&V page su2code/su2code.github.io#77

@pcarruscag pcarruscag merged commit 55b2723 into develop May 31, 2022
@pcarruscag pcarruscag deleted the swbli branch May 31, 2022 16:41
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.

4 participants