Skip to content

Artificial viscosity#213

Closed
w-hagen wants to merge 111 commits intoillinois-ceesd:mainfrom
w-hagen:artificial_viscosity
Closed

Artificial viscosity#213
w-hagen wants to merge 111 commits intoillinois-ceesd:mainfrom
w-hagen:artificial_viscosity

Conversation

@w-hagen
Copy link
Copy Markdown
Collaborator

@w-hagen w-hagen commented Feb 3, 2021

This branch implements a modal indicator based artificial viscosity approach to enable shock capturing.

Outstanding work needed before moving forward:

  • Clean up boundary condition routines utilized for the artificial viscosity
  • Expose smoothness indicator parameters
  • Remove unnecessary make_obj_array calls
  • Bring branch up to master
  • Add documentation of approach
  • Add tests for additions

@MTCam
Copy link
Copy Markdown
Member

MTCam commented May 6, 2021

CI failures due to inducer/grudge#83 moving stuff around. @thomasgibson. Fixed these in NS with:
(e451ea4, e4cb256).

@MTCam I'll take care of these updates and I'll file a mirgecom PR and one against @w-hagen's branch

Thanks a ton, @thomasgibson!

@MTCam
Copy link
Copy Markdown
Member

MTCam commented May 7, 2021

Hey... the (approve and run) button went away. 🎉

@MTCam MTCam mentioned this pull request May 10, 2021
majosm and others added 3 commits May 10, 2021 19:33
* remove call to deprecated create_parallel_grid

* remove use of deprecated n parameter

* add boundary_kwargs to av_operator

* raise exception instead of asserting
MTCam added a commit that referenced this pull request May 13, 2021
Comment thread mirgecom/artificial_viscosity.py Outdated
@MTCam
Copy link
Copy Markdown
Member

MTCam commented Jun 25, 2022

Closing this as stale. We merged AV in #582

@MTCam MTCam closed this Jun 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request production-critical On the critical path for CEESD production capability.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants