Skip to content

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

Open

Description

@jgabry

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.

Activity

  1. LokeshShukla910 commented on Feb 7, 2026

    @LokeshShukla910

    I have pulled a commit request after solving the issue. Please review it and let me know if it need some changes

  2. added a commit that references this issue on Feb 24, 2026
    ed719d3
  3. utkarshpawade commented on Feb 24, 2026

    @utkarshpawade
    Contributor

    Hi @jgabry,

    I've created PR #423 to address this issue. Since the previous PR appeared to be stalled, I've implemented the fix with the following changes:

    • Removed y-axis line from all density, histogram, dot, violin, and frequency polygon plots (13 functions total across mcmc_*, ppc_*, and ppd_* families)
    • Added theme(axis.line.y = element_blank()) following the pattern already used in mcmc_dots()
    • Updated all visual regression test snapshots
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions