Skip to content

gh-256: inversion - #303

Merged
JaimeRZP merged 47 commits into
mainfrom
256-inversion
Sep 16, 2025
Merged

gh-256: inversion#303
JaimeRZP merged 47 commits into
mainfrom
256-inversion

Conversation

@JaimeRZP

@JaimeRZP JaimeRZP commented Jul 15, 2025

Copy link
Copy Markdown
Contributor

Implements the methods to invert the mixing matrix
#256

@codecov

codecov Bot commented Jul 15, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Flag Coverage Δ
py3.10 83.89% <100.00%> (+0.27%) ⬆️
py3.11 83.89% <100.00%> (+0.27%) ⬆️
py3.12 83.86% <100.00%> (+0.27%) ⬆️
py3.13 83.86% <100.00%> (+0.27%) ⬆️
py3.9 83.92% <100.00%> (+0.27%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
heracles/__init__.py 82.35% <ø> (ø)
heracles/twopoint.py 95.75% <100.00%> (+1.14%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JaimeRZP JaimeRZP mentioned this pull request Jul 15, 2025
@JaimeRZP JaimeRZP self-assigned this Sep 9, 2025
@JaimeRZP JaimeRZP added the enhancement New feature or request label Sep 9, 2025
@JaimeRZP JaimeRZP changed the title gh-256 inversion gh-256: inversion Sep 9, 2025
@JaimeRZP

JaimeRZP commented Sep 9, 2025

Copy link
Copy Markdown
Contributor Author

It seems like rtol is broken in python3.9
We also have to come up with more tests... but this is testing everything already...

Comment thread heracles/twopoint.py Outdated
JaimeRZP and others added 3 commits September 16, 2025 10:00
Good spot

Co-authored-by: Nicolas Tessore <n.tessore@ucl.ac.uk>
@JaimeRZP
JaimeRZP merged commit f7a7c26 into main Sep 16, 2025
10 checks passed
@JaimeRZP
JaimeRZP deleted the 256-inversion branch September 16, 2025 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants