stan-dev / stan-dev/rstantools
reloo generic
Nobody has claimed this yet.
- Dominant language
- R
- Stars
- 48
- Forks
- 24
- Avg merge
- 1h 17m
- Merged PRs (30d)
- 1
Description
@paul-buerkner I'd like to add a generic for reloo so we can have reloo.stanreg and reloo.brmsfit methods. But I see that currently brms takes the first argument of reloo to be the loo object. So it can't change to a brmsfit method without breaking backwards compatibility, right? If that's the case then maybe we should use a different name than reloo for the generic and new methods and you can deprecate reloo it in brms. What do you think? Definitely open to other ideas.
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
The issue names only the reloo API and its brms argument order; begin by locating current reloo callers and method definitions. Confirm whether stanreg and brmsfit methods can be introduced without breaking existing calls, or document an agreed replacement and deprecation plan. Done means the compatibility decision is settled.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- r
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100