Skip to content

Change ts_map default MOC refinement strategy#508

Open
jdbuhler wants to merge 4 commits intocositools:developfrom
McKelvey-Engineering-CSE:tsmap-fix
Open

Change ts_map default MOC refinement strategy#508
jdbuhler wants to merge 4 commits intocositools:developfrom
McKelvey-Engineering-CSE:tsmap-fix

Conversation

@jdbuhler
Copy link
Contributor

To address issue #465 , this patch changes the default pruning strategy used by MOC ts_map from top-k to containment-region-based. The top-k strategy is prone to implementation-dependent behavior when many pixels have the same likelihood (usually 0). The first test case for MOC ts_map has been updated to reflect the new default.

I incidentally fixed a couple of syntax warnings in fast_norm_fit.py caused by backslashes in docstrings.

@codecov
Copy link

codecov bot commented Mar 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.18%. Comparing base (4911568) to head (42e7b71).

Files with missing lines Coverage Δ
cosipy/ts_map/fast_norm_fit.py 49.50% <ø> (ø)
cosipy/ts_map/moc_ts_fit.py 97.89% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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