Skip to content

Redesigning output system#1023

Merged
andrsd merged 4 commits intomainfrom
exo-refact
Feb 24, 2026
Merged

Redesigning output system#1023
andrsd merged 4 commits intomainfrom
exo-refact

Conversation

@andrsd
Copy link
Owner

@andrsd andrsd commented Feb 24, 2026

  • Moved code from ExodusIIOutput into a new IO module
  • Problems need to override output_with and update the solution vector so that the output saves the up-to-date values

@andrsd andrsd added this to the v0.14 milestone Feb 24, 2026
@andrsd andrsd self-assigned this Feb 24, 2026
@andrsd andrsd added the api-change PR changes API, i.e. it will break depending apps label Feb 24, 2026
@codecov
Copy link

codecov bot commented Feb 24, 2026

Codecov Report

❌ Patch coverage is 75.05198% with 120 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.30%. Comparing base (de3ab3a) to head (99757c9).
⚠️ Report is 5 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/IO.cpp 81.31% 74 Missing ⚠️
src/ExodusIIOutput.cpp 45.00% 33 Missing ⚠️
src/ExplicitDGLinearProblem.cpp 0.00% 4 Missing ⚠️
src/ExplicitFVLinearProblem.cpp 0.00% 4 Missing ⚠️
src/FEProblemInterface.cpp 0.00% 3 Missing ⚠️
src/TecplotOutput.cpp 71.42% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1023      +/-   ##
==========================================
+ Coverage   75.91%   79.30%   +3.38%     
==========================================
  Files         162      163       +1     
  Lines       10571    10606      +35     
==========================================
+ Hits         8025     8411     +386     
+ Misses       2546     2195     -351     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@andrsd andrsd merged commit fafe4eb into main Feb 24, 2026
7 of 8 checks passed
@andrsd andrsd deleted the exo-refact branch February 24, 2026 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api-change PR changes API, i.e. it will break depending apps

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant