Skip to content
Snippets Groups Projects

BUGFIX: make sure chain tracks all points

Merged Colm Talbot requested to merge bugfix-act-sampling into master

It looks like two bugs crept into !1187 (merged) just before it was merged.

  • one is the ordering of the updates of the bounding ellipses and the MCMC length for the LivePointMultiEllipsoidSampler. The fix here means that the scale attribute will update as expected within dynesty.
  • the ACT tracking broke because points weren't actually being added to the chain during the loop.

I also removed an excess full ACT calculation as nothing is bigger than inf!

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading