Consider changing default behavior of pathfinder num_paths=1
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 238
- Forks
- 106
- Avg merge
- 2h 29m
- Merged PRs (30d)
- 6
Description
When pathfinder was first being exposed, it was decided that num_paths=1 would run single-path pathfinder. This was at the time the only way to turn off PSIS resampling.
Now that we have the argument psis_resample, we could do this another way, but that requires changing the default. In todays Stan The Gathering, @avehtari @mitzimorris and @bob-carpenter all voted in favor of making this change
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Begin with the pathfinder command-line handling of num_paths=1 and psis_resample. Review the current default behavior and its existing documentation or tests, then clarify the intended default before making changes. Done means the agreed default is implemented consistently and its behavior is covered.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- cli
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100