Skip to content

Apply the v1.16.1 fixes to master - #692

Merged
marcpaterno merged 6 commits into
masterfrom
apply-v1.16.1-fixes-to-master
Oct 23, 2026
Merged

marcpaterno merged 6 commits into
masterfrom
apply-v1.16.1-fixes-to-master

Conversation

@vitenti

@vitenti vitenti commented Oct 23, 2026 •

Copy link
Copy Markdown
Collaborator

@github-actions

github-actions Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

PR revision: 640d1c355b2d88c5603547dbe14abad357c9dc56
Target: master

No breaking API changes detected.

PR public API verdict

No breaking API changes detected.

Target: master
Revision: 640d1c355b2d88c5603547dbe14abad357c9dc56
Baseline: 2568be3d05bacb21f35a47a878e6556c5f99795b

Static analysis only: no behavioral compatibility guarantee or external-dependency signature analysis.

Public API changes: 2568be3 -> HEAD

Executive summary

No breaking API changes detected.

Breaking changes

None.

New public API

firecrown.likelihood.StudentT

  • firecrown.likelihood.StudentT.make_realization_vector

firecrown.likelihood_base

  • firecrown.likelihood_base.ABC

Complete report artifact and workflow run | API job

@vitenti
vitenti requested a review from marcpaterno October 23, 2026 02:48
@codecov

codecov Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.0%. Comparing base (e2eed06) to head (640d1c3).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #692   +/-   ##
=======================================
  Coverage   100.0%   100.0%           
=======================================
  Files         163      163           
  Lines        9977     9978    +1     
  Branches     1199     1199           
=======================================
+ Hits         9977     9978    +1     
Files with missing lines Coverage Δ
firecrown/app/analysis/_numcosmo.py 100.0% <100.0%> (ø)
firecrown/app/examples/_cluster_data.py 100.0% <100.0%> (ø)
firecrown/likelihood/_binned_cluster.py 100.0% <100.0%> (ø)
firecrown/likelihood/_gaussfamily.py 100.0% <100.0%> (ø)
firecrown/likelihood/_student_t.py 100.0% <100.0%> (ø)
firecrown/likelihood_base.py 100.0% <100.0%> (ø)
firecrown/models/two_point/_theory.py 100.0% <100.0%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

vitenti and others added 6 commits October 23, 2026 09:01
* Build the cosmology with prim= and reion= in _numcosmo.py instead of add_submodel
* Fix Yp with NcBBNParametrized in the test fixtures and call set_z_from_tau(tau)
* Pass submodels to the constructors in the remaining NumCosmo tests and cluster examples
* Raise the numcosmo pin to >=0.28,<1.0 in dependencies.yaml, environment.yml and pyproject.toml
* Pin CosmoSIS to `<3.26` because 3.26 ships Fortran module files built with gfortran 15, while the current `camb <2.0` constraint keeps Firecrown's build environment on gfortran 14
* Update intermediate likelihood and statistic classes for pylint 4.1 abstract-class detection
* Move `SourceGalaxy.tracer_args` annotation to the class body so pylint recognizes the attribute declaration
* Override unsupported `StudentT.make_realization_vector` behavior explicitly and add coverage for it
* Replace the incomplete likelihood test helper with a concrete implementation while testing the base default directly
* Normalize SciPy count and bin-edge outputs to `float64` for scipy-stubs compatibility
* Verify mypy, pylint, black, flake8, affected tests, and the CosmoSIS standard-library build
* Compute `min_ell` and `max_ell` in `TwoPointTheory.generate_ells_for_interpolation` with `np.min`/`np.max` instead of `self.ells[0]`/`self.ells[-1]`
* Fix pylint E1136 (unsubscriptable-object) on `self.ells`, reported when the pylint worker checking `_theory.py` has not analyzed `firecrown/likelihood/_two_point.py`, where `ells` is assigned
* Re-solve the lockfiles with master's manifest: the v1.16.1 versions plus griffe from #680
* Regenerate environment.yml with pygobject-stubs <2.18
* Create HICosmoDEXcdm with prim and reion in _cluster_data._numcosmo_cosmology instead of attaching them with add_submodel, deprecated in NumCosmo 0.28
@vitenti
vitenti force-pushed the apply-v1.16.1-fixes-to-master branch from 092db9a to 640d1c3 Compare October 23, 2026 12:11
@marcpaterno
marcpaterno merged commit 084427d into master Oct 23, 2026
16 checks passed
@marcpaterno
marcpaterno deleted the apply-v1.16.1-fixes-to-master branch October 23, 2026 13:33
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.

2 participants