Skip to content

Exclude tests involving adjoint solver from make check#3181

Open
oskooi wants to merge 3 commits intoNanoComp:masterfrom
oskooi:make_check_exclude_adjoint
Open

Exclude tests involving adjoint solver from make check#3181
oskooi wants to merge 3 commits intoNanoComp:masterfrom
oskooi:make_check_exclude_adjoint

Conversation

@oskooi
Copy link
Copy Markdown
Collaborator

@oskooi oskooi commented Mar 26, 2026

The four tests involving the adjoint solver (particularly test_adjoint_solver.py) take an unusually long time to complete which tends to slow down the CI. This PR modifies make check to exclude these tests and instead creates a make check-adjoint whicn can be used to run all tests (including those involving the adjoint solver).

@codecov-commenter
Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.65%. Comparing base (f29a8c7) to head (34cdcde).
⚠️ Report is 117 commits behind head on master.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

❗ There is a different number of reports uploaded between BASE (f29a8c7) and HEAD (34cdcde). Click for more details.

HEAD has 4 uploads less than BASE
Flag BASE (f29a8c7) HEAD (34cdcde)
5 1
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #3181       +/-   ##
===========================================
- Coverage   73.81%   63.65%   -10.16%     
===========================================
  Files          18       18               
  Lines        5423     5454       +31     
===========================================
- Hits         4003     3472      -531     
- Misses       1420     1982      +562     

see 10 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@stevengj
Copy link
Copy Markdown
Collaborator

stevengj commented Apr 3, 2026

Is there a way to conditionally run this?

And/or run it periodically or upon a trigger?

@stevengj
Copy link
Copy Markdown
Collaborator

stevengj commented Apr 3, 2026

Maybe add a workflow for a new action to run make check-adjoint periodically (e.g. once a week), which we can also trigger manually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants