Skip to content

Jacobian and stabilization improvement#32836

Open
lingzou wants to merge 1 commit intoidaholab:nextfrom
lingzou:Jacobian-and-stabilization-improvement
Open

Jacobian and stabilization improvement#32836
lingzou wants to merge 1 commit intoidaholab:nextfrom
lingzou:Jacobian-and-stabilization-improvement

Conversation

@lingzou
Copy link
Copy Markdown
Contributor

@lingzou lingzou commented Apr 24, 2026

Closes #32835

Reason

See issue #32835

Design

See issue #32835

Impact

See issue #32835

@lingzou
Copy link
Copy Markdown
Contributor Author

lingzou commented Apr 24, 2026

Clearly something I have messed up from seeing thousands of 'commits.'

@loganharbour
Copy link
Copy Markdown
Member

You rebased/based your PR off of the master branch. You should base it off of the devel branch.

The laziest fix you can do here:

git fetch upstream
git reset --hard upstream/devel
git cherry-pick d8101d514b8c3947de10b67c54d02f7a396957f1
git cherry-pick 9064a411130543677af50b3ad9d30aee695030f4
git push -f

This will reset to devel and then pull in your two commits.

@moosebuild
Copy link
Copy Markdown
Contributor

moosebuild commented Apr 25, 2026

Job Documentation, step Docs: sync website on 5e77e9f wanted to post the following:

View the site here

This comment will be updated on new commits.

@lingzou lingzou force-pushed the Jacobian-and-stabilization-improvement branch from 9064a41 to 972ad7c Compare April 25, 2026 16:38
@GiudGiud GiudGiud self-assigned this Apr 30, 2026
@lingzou lingzou force-pushed the Jacobian-and-stabilization-improvement branch from 972ad7c to 5e77e9f Compare April 30, 2026 21:10
@moosebuild
Copy link
Copy Markdown
Contributor

Job Test, step Results summary on 5e77e9f wanted to post the following:

Framework test summary

Compared against 4e53b3b in job civet.inl.gov/job/3786076.

No change

Modules test summary

Compared against 4e53b3b in job civet.inl.gov/job/3786076.

No change

@moosebuild
Copy link
Copy Markdown
Contributor

Job Coverage, step Generate coverage on 5e77e9f wanted to post the following:

Framework coverage

4e53b3 #32836 5e77e9
Total Total +/- New
Rate 85.87% 85.87% +0.00% -
Hits 132516 132517 +1 0
Misses 21798 21797 -1 0

Diff coverage report

Full coverage report

Modules coverage

Navier stokes

4e53b3 #32836 5e77e9
Total Total +/- New
Rate 85.77% 85.79% +0.02% 100.00%
Hits 21110 21145 +35 35
Misses 3502 3502 - 0

Diff coverage report

Full coverage report

Full coverage reports

Reports

This comment will be updated on new commits.

Copy link
Copy Markdown
Contributor

@GiudGiud GiudGiud left a comment

Choose a reason for hiding this comment

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

could you please check in a Jacobian check test for conservative and non-conservative form?
With the PETScJacobianChecker

looks good otherwise


tau_base = std::min(_hsupg[_qp] / (2 * _vel_mag), _dt / 2);
_taum[_qp] = std::clamp(_taum[_qp], 0.01 * tau_base, 2.0 * tau_base);
_taue[_qp] = std::clamp(_taue[_qp], 0.01 * tau_base, 2.0 * tau_base);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this should likely be added to the explanation of SUPG now

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.

Navier-Stokes CG-FEM minor improvements

5 participants