Skip to content
Snippets Groups Projects

Adding an ACT estimate for the walks

Merged Gregory Ashton requested to merge add-act-estimated-walks into master
  • Improvements to the bilby implemented rwalk method

    • rwalk a new implementation which calculates an ACT to better estimate the number of steps needed
  • The rwalk sample method now defaults to bilby.core.sample.dynesty.sample_rwalk_bilby_with_act which adds the maxmcmc, nact. These control the maximum number of walks, the number of ACT required before accepting a point.

  • Uses the CPNest method to calculate the ACT

  • Adds a guide to the documentation on using dynesty

  • Updates the default vol_dec to 8 and enlarge to 1.5 which was found to improve boundary behaviour Note: during development, I experimented with adding a Jacobian and adaptive walks. I found in practise these didn't help sampling so I've removed them for simplicity.

  • Updates the walks to default to 100. This is now the minimum number of steps.

Edited by Gregory Ashton

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
  • Gregory Ashton resolved all threads

    resolved all threads

  • Gregory Ashton added 1 commit

    added 1 commit

    • 3babc722 - Only print warning once and only break of accept > 0

    Compare with previous version

  • Gregory Ashton unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Gregory Ashton added 2 commits

    added 2 commits

    • 4aeb56f7 - Flatten the walking loop to avoid bias and add the rstagger method
    • 84676190 - Merge branch 'add-act-estimated-walks' of git.ligo.org:Monash/tupak into add-act-estimated-walks

    Compare with previous version

  • Gregory Ashton added 1 commit

    added 1 commit

    Compare with previous version

  • Gregory Ashton added 1 commit

    added 1 commit

    Compare with previous version

  • Gregory Ashton changed the description

    changed the description

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading