CI: don't cache amici; pytest ignore amici_models; passenv BNGPATH

[dweindl]

• Don't cache the amici installation. We are using amici/develop on purpose to detect problems early on. E.g., test_deepcopy_objective fails #1520 should have been detected much earlier.
• Ignore amici_models when collecting tests
• Use external BNGPATH if set

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.79%. Comparing base (6eb4245) to head (b647f53).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #1523      +/-   ##
===========================================
- Coverage    82.83%   82.79%   -0.04%     
===========================================
  Files          163      163              
  Lines        13962    13962              
===========================================
- Hits         11565    11560       -5     
- Misses        2397     2402       +5     

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

[PaulJonasJost]

What do we need a BNGPATH for? It is for bionetgen but I am not entirely aware here we use it.

Independent, should be a good improvement, thanks 👍🏼

[dweindl]

What do we need a BNGPATH for? It is for bionetgen but I am not entirely aware here we use it.

Yes, for BioNetGen, which is used by pysb.

Also used here:

pyPESTO/test/petab/test_petab_import.py

Lines 23 to 29 in 3b97e34

	# In CI, bionetgen is installed here
	BNGPATH = os.path.abspath(
	os.path.join(os.path.dirname(__file__), "..", "..", "BioNetGen-2.8.5")
	)
	if "BNGPATH" not in os.environ:
	logging.warning(f"Env var BNGPATH was not set. Setting to {BNGPATH}")
	os.environ["BNGPATH"] = BNGPATH