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

Fixes needed for Manifold Projection Semi-Explicit DAEs #1037

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

avik-pal
Copy link
Member

I am using this to accumulate all the fixes needed for the project, so let's hold off on merging.

  • Fixes SteadyState Adjoint for OOP jacobian

Copy link

codecov bot commented May 15, 2024

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 58.38%. Comparing base (eacd462) to head (d75fefa).

❗ Current head d75fefa differs from pull request most recent head 3eaf6f0. Consider uploading reports for the commit 3eaf6f0 to get more accurate results

Files Patch % Lines
src/steadystate_adjoint.jl 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1037      +/-   ##
==========================================
- Coverage   60.77%   58.38%   -2.40%     
==========================================
  Files          19       19              
  Lines        4535     4534       -1     
==========================================
- Hits         2756     2647     -109     
- Misses       1779     1887     +108     

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

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.

1 participant