stan-dev / stan-dev/bayesplot

Remove y-axis for plots where y-axis has no purpose

Open
#418 2 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

good first issue plot aesthetics
Dominant language
R
Stars
442
Forks
93
Avg merge
3h 11m
Merged PRs (30d)
1

Description

This would technically break visual backwards compatibility although wouldn't affect anyone's code. @avehtari pointed out that for some plots we have a y-axis that serves no purpose and the plots look better without it. For example, the new mcmc_dots() function doesn't include the y-axis for this reason, but functions like mcmc_dens() and others include the y-axis for no reason.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by comparing the y-axis treatment in mcmc_dots() with mcmc_dens() and the other plotting functions mentioned in the issue. Identify plots where the y-axis has no purpose, then verify that those plots render without it while the requested visual change does not affect code usage.

Written by the indexing model from the issue text.

Assessment

Tech stack
r
Domain
data-visualization
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.