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

Revert "Condensed eigen system refactor" #28624

Merged
merged 1 commit into from
Sep 16, 2024

Conversation

lindsayad
Copy link
Member

@lindsayad lindsayad commented Sep 15, 2024

Reverts #28452

This refactor didn't capture when the mesh changes

@lindsayad lindsayad marked this pull request as ready for review September 15, 2024 18:49
@lindsayad
Copy link
Member Author

lindsayad commented Sep 15, 2024

Opened libMesh/libmesh#3950 after whose conclusion we can revert this revert

@lindsayad lindsayad force-pushed the revert-28452-condensed-eigen-system-27262-refactor branch from 0e5ba32 to e3efe51 Compare September 15, 2024 19:02
@lindsayad
Copy link
Member Author

fyi @YaqiWang

@moosebuild
Copy link
Contributor

Job Documentation on e3efe51 wanted to post the following:

View the site here

This comment will be updated on new commits.

@moosebuild
Copy link
Contributor

Job Coverage on e3efe51 wanted to post the following:

Framework coverage

8cd741 #28624 e3efe5
Total Total +/- New
Rate 85.04% 85.04% +0.00% 95.65%
Hits 105618 105635 +17 22
Misses 18582 18582 - 1

Diff coverage report

Full coverage report

Modules coverage

Coverage did not change

Full coverage reports

Reports

This comment will be updated on new commits.

@lindsayad lindsayad merged commit d13cb8d into next Sep 16, 2024
49 of 50 checks passed
@lindsayad lindsayad deleted the revert-28452-condensed-eigen-system-27262-refactor branch September 16, 2024 02:26
@YaqiWang
Copy link
Contributor

Thanks for the fix.

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.

4 participants