Allow chains argument when algorithm is not sampling
Nobody has claimed this yet.
- Dominant language
- R
- Stars
- 401
- Forks
- 136
- PR merge metrics
- No merged PRs in 30d
Description
I usually start scripting my analyses with other algorithms, such as meanfield, make sure the script is running smoothly and without errors, and only then replace meanfield with "sampling" and run the analysis (which could take then days to compute).
Currently, setting the chains parameter if algorithm="meanfield" returns an error (Error: passing unknown arguments: chains.), leading to adding and removing of all the chains parameters within a script everytime we switch between two algorithms, which can become tedious in the case of many models.
I would be nice if the chains argument would just be ignored if not relevant (there could be a message: "chains parameter ignored"). What are your opinions on this?
Contributor guide
No contributing guide indexed for this repository
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
Start by locating the argument validation for chains and compare its behavior when algorithm is "meanfield" versus "sampling". Done means scripts can retain chains while using meanfield without an unknown-argument error, while chains continues to work for sampling.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- r
- Domain
- data
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100