Make most plotting functions methods
Nobody has claimed this yet.
- Dominant language
- R
- Stars
- 442
- Forks
- 93
- Avg merge
- 3h 11m
- Merged PRs (30d)
- 1
Description
I think it would help usability to make the various functions methods. I love that the functions are frontend agnostic and take arrays or data frames, but it would make it much more usable if the they were methods, so it would work directly with stanfit and stanreg objects.
I haven't looked too much at the code, but it looks like the internal function prepare_mcmc_array is doing most of what a method dispatcher would do. Maybe just exposing that function and making it a method would be sufficient.
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
Start by reviewing the plotting functions and the internal prepare_mcmc_array function mentioned in the issue, along with how stanfit and stanreg objects are represented. Determine the scope of method dispatch while preserving support for arrays and data frames. Done means the relevant plotting functions work directly with stanfit and stanreg objects without losing their frontend-agnostic inputs.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- r
- Domain
- data-visualization
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100