Skip to content

Conversation

@A-acuto
Copy link
Contributor

@A-acuto A-acuto commented Feb 28, 2025

This PR is built on top of these PRs #1080 and #1131 :

The No-U Turn sampler (NUTS) is implemented as Proposal subclass. In this gist there is a simple comparison between a particle filter (DynamicsProposal) implementation and NUTSProposal https://gist.github.com/A-acuto/677df96b2efebaef15ab51f7a616f5d2 (still vaid)

This PR is part of the Fusion 2025 paper submission.
The latest changes include a better factoring of the NUTS code, some functions to help it easier to debug and improve, as well as better testing.

@A-acuto A-acuto closed this Oct 1, 2025
@A-acuto A-acuto reopened this Oct 1, 2025
@codecov
Copy link

codecov bot commented Oct 1, 2025

Codecov Report

❌ Patch coverage is 35.78431% with 131 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.25%. Comparing base (1cdbf72) to head (df11d70).
⚠️ Report is 202 commits behind head on main.

Files with missing lines Patch % Lines
stonesoup/proposal/nuts.py 35.78% 125 Missing and 6 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1133      +/-   ##
==========================================
- Coverage   93.99%   93.25%   -0.74%     
==========================================
  Files         227      228       +1     
  Lines       15771    15975     +204     
  Branches     2179     2191      +12     
==========================================
+ Hits        14824    14898      +74     
- Misses        659      784     +125     
- Partials      288      293       +5     
Flag Coverage Δ
integration 65.65% <0.00%> (-0.87%) ⬇️
unittests 90.71% <35.78%> (-0.73%) ⬇️

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

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

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

@A-acuto A-acuto marked this pull request as ready for review October 1, 2025 12:40
@A-acuto A-acuto requested a review from a team as a code owner October 1, 2025 12:40
@A-acuto A-acuto requested review from hpritchett-dstl and sdhiscocks and removed request for a team October 1, 2025 12:40
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