Skip to content

Conversation

@alexanderianblair
Copy link
Collaborator

Closes #31595.

@alexanderianblair alexanderianblair changed the title Alexanderianblair/mfem multivariate sum Extend MFEMSumAux to sum over an arbitrary number of MFEMVariables Sep 24, 2025
@moosebuild
Copy link
Contributor

moosebuild commented Sep 25, 2025

Job Documentation, step Docs: sync website on 44238c2 wanted to post the following:

View the site here

This comment will be updated on new commits.

@moosebuild
Copy link
Contributor

moosebuild commented Sep 25, 2025

Job Coverage, step Generate coverage on 44238c2 wanted to post the following:

Framework coverage

a1e99f #31599 44238c
Total Total +/- New
Rate 85.91% 85.91% +0.00% 100.00%
Hits 122646 122650 +4 17
Misses 20120 20120 - 0

Diff coverage report

Full coverage report

Modules coverage

Coverage did not change

Full coverage reports

Reports

This comment will be updated on new commits.

@alexanderianblair alexanderianblair force-pushed the alexanderianblair/mfem-multivariate-sum branch from 4696608 to ab98787 Compare September 25, 2025 12:38
@alexanderianblair alexanderianblair marked this pull request as ready for review September 25, 2025 14:20
@alexanderianblair alexanderianblair changed the title Extend MFEMSumAux to sum over an arbitrary number of MFEMVariables WIP: Extend MFEMSumAux to sum over an arbitrary number of MFEMVariables Sep 26, 2025
@alexanderianblair alexanderianblair changed the title WIP: Extend MFEMSumAux to sum over an arbitrary number of MFEMVariables Extend MFEMSumAux to sum over an arbitrary number of MFEMVariables Sep 30, 2025
@alexanderianblair
Copy link
Collaborator Author

Should now be ready for review - was briefly moved back to 'WIP' whilst an issue observed with multiple MFEMCutTransitionSubMesh objects in multiapps when using 'clone_parent_mesh=true'. Now confirmed that issue is unrelated; a fix for the latter problem should be incoming shortly

Copy link
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.

looks good!
in MOOSE we called that one the LinearCombination... (Function probably) but Sum makes more sense

@alexanderianblair alexanderianblair force-pushed the alexanderianblair/mfem-multivariate-sum branch from ab98787 to 1643c99 Compare October 2, 2025 11:06
@lindsayad
Copy link
Member

failure is unrelated

@lindsayad lindsayad merged commit 3196b12 into idaholab:next Oct 3, 2025
68 of 69 checks passed
nmnobre added a commit to nmnobre/moose that referenced this pull request Nov 2, 2025
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.

Support summation over arbitrary numbers of MFEM variables

4 participants