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

add a new Eulerian supersonic flow around a cylinder case #633

Merged
merged 5 commits into from
Aug 18, 2024

Conversation

ZhentongWang
Copy link
Collaborator

@ZhentongWang ZhentongWang commented Aug 8, 2024

output_velocity_LG
output_pressure_LG

The velocity and pressure contours of the Eulerian supersonic flow around a cylinder using the Laguerre-Gauss kernel are presented.

The main changes are as follows:

  1. Two new files, "eulerian_ghost_boundary.h" and "eulerian_ghost_boundary.cpp", are created for adding a new boundary for Eulerian cases, similar with the FVM boundary conditions, based on the zero-order consistency.
  2. The classes for creating the ghost particles and the boundary conditions for boundary particles in Eulerian SPH method are introduced.
  3. One class, namely "GhostKernelGradientUpdate", is used for updating the dW_ig and e_ig for the ghost particles to obey the zero-order consistency strictly after implementing the kernel correction.
  4. The different approach for calculating the smallest and largest wave speeds in the HLLC Riemann solver is also introduced.

@Xiangyu-Hu
Copy link
Owner

Xiangyu-Hu commented Aug 8, 2024

@ZhentongWang could you also put some figures on the results and brief summary on the changes?

@ZhentongWang
Copy link
Collaborator Author

@ZhentongWang could you also put some figures on the results and brief summary on the changes?

Sure, I have added the pressure and velocity contours of this case, as well as the modification summary.

@ZhentongWang
Copy link
Collaborator Author

@ZhentongWang could you also put some figures on the results and brief summary on the changes?

Sure, I have added the pressure and velocity contours of this case, as well as the modification summary.

please see the description comments.

@Xiangyu-Hu
Copy link
Owner

@ZhentongWang also put the updated results?

@ZhentongWang
Copy link
Collaborator Author

@ZhentongWang also put the updated results?

I have updated the velocity and pressure contours.

@Xiangyu-Hu
Copy link
Owner

@ZhentongWang the picture is so small.

@ZhentongWang
Copy link
Collaborator Author

@ZhentongWang the picture is so small.

The contours have been enlarged.

Copy link
Collaborator

@FengWang3119 FengWang3119 left a comment

Choose a reason for hiding this comment

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

Just curious about the naming, such as rhol_cl, rhor_cr,clr.
And I am confused about the case_name in the regression_test_tool.py,"test_1d_shock_tube"?

@ZhentongWang
Copy link
Collaborator Author

Just curious about the naming, such as rhol_cl, rhor_cr,clr. And I am confused about the case_name in the regression_test_tool.py,"test_1d_shock_tube"?

Thanks for pointing out. l and r means left and right states in the Riemann solver. Here, rhol_cl means rho times sound speed in left state, just like rho0c0_i_ meaning reference rho times reference sound speed of the particle I in the acoustic Riemann solver.

Besides, I have modified the names in the regression file.

@ZhentongWang ZhentongWang merged commit 3ed7659 into master Aug 18, 2024
7 checks passed
@Xiangyu-Hu Xiangyu-Hu deleted the eulerian_supersonic_flow_with_new_BC branch August 19, 2024 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants