Skip to content

Cont damage fix - #1723

Draft
ChrisZYJ wants to merge 14 commits into
MFlowCode:masterfrom
ChrisZYJ:cont_damage_fix
Draft

Cont damage fix#1723
ChrisZYJ wants to merge 14 commits into
MFlowCode:masterfrom
ChrisZYJ:cont_damage_fix

Conversation

@ChrisZYJ

Copy link
Copy Markdown
Contributor

Description

Bug fix; closes #1673.

In progress (almost done).

Testing

How did you test your changes?

Checklist

Check these like this [x] to indicate which of the below applies.

  • I added or updated tests for new behavior
  • I updated documentation if user-facing behavior changed

See the developer guide for full coding standards.

GPU changes (expand if you modified src/simulation/)
  • GPU results match CPU results
  • Tested on NVIDIA GPU or AMD GPU

AI code reviews

Reviews are not retriggered automatically. To request a review, comment on the PR:

  • @claude full review — Claude full review (also triggers on PR open/reopen/ready)
  • Or add label claude-full-review — Claude full review via label

@ChrisZYJ

Copy link
Copy Markdown
Contributor Author

Almost done. Will update very soon.

@github-actions

Copy link
Copy Markdown

Lines of Code

File Lines Diff
src/simulation/m_hypoelastic.fpp 627 +37
src/simulation/m_data_output.fpp 1349 +32
src/simulation/m_riemann_solver_hll.fpp 601 +20
src/common/m_variables_conversion.fpp 1015 +18
src/simulation/m_riemann_solver_hllc.fpp 1288 +16
src/simulation/m_time_steppers.fpp 830 +2
Directory Lines Diff
common 9796 +18
simulation 28125 +107
total 45754 +125

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Continuum-damage model has transport, stress-measure, principal-stress, and documentation inconsistencies

2 participants