Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Meta-geom: Make a single stat_slabinterval / geom_slabinterval from the hacked violins/densities, pointinterval, and eye geoms #84

Closed
41 tasks done
mjskay opened this issue Oct 16, 2017 · 1 comment
Milestone

Comments

@mjskay
Copy link
Owner

mjskay commented Oct 16, 2017

TODOs:

  • add a few more shortcuts (and update the family overview image):
    • stat_dist_interval[h]
    • stat_dist_pointinterval[h]
    • stat_slab[h]
    • stat_dist_slab[h]
    • geom_slab[h] ?
  • tests please:
    • break functions out so can see coverage
    • tests for scale transformations in stats with things like scale_x_log10()
  • add examples to new geom docs (search for TODO). Can pull from slabinterval vignette. Then point people in the vignette direction from the examples
  • output n so people can use n*density if desired
  • add a few more of the specific aesthetics (plus test):
    • point_alpha
    • interval_alpha
    • slab_fill
    • slab_alpha
  • reference in README, other vignettes (even if just pointing to slabinterval vignette)
  • document aesthetics for geom_slabinterval and sub-geoms
  • change old geom_[half]eye[h] refs to stat_[half]eye[h]
  • speed up gradient by using polygon directly for groups
  • make thickness a different stat
  • build cool new stuff the meta-geom makes easy:
  • build ridiculous meta-geom
  • decide on good name
  • implement a key / legend drawing function
    • remove old draw_key funcs
  • look into dropping ggstance stuff in docs (except dodgev)
  • document use of position dodge
  • reimplement base geoms (not stats) on meta-geom
  • undo weird interval_size scale and revert to fatten / etc
  • fix existing stats
  • build generic meta-stat (and make it faster)
    • rebuild stat_pointinterval and stat_interval on top of this
  • add in depracted argument alias for relative_scale -> scale
  • fix behavior of position_dodge for pointintervals / eyes / halfeyes (it now works for intervals, see Make stat_interval provide width automatically for position='dodge' #180)
  • Make sure aesthetics for different parts of the geom are exposed (fill, alpha, outline, etc)
  • make sure defaults only work when corresponding column exists; see failure of geom_pointinterval in readme and make a test.
@mjskay mjskay self-assigned this Oct 16, 2017
@mjskay mjskay changed the title Merge the hacked violins/densities and eye geoms Make a single stat_density_interval / geom_density_interval from the hacked violins/densities, pointinterval, and eye geoms Aug 10, 2018
@mjskay mjskay removed their assignment Aug 10, 2018
@mjskay mjskay changed the title Make a single stat_density_interval / geom_density_interval from the hacked violins/densities, pointinterval, and eye geoms Meta-geom: Make a single stat_density_interval / geom_density_interval from the hacked violins/densities, pointinterval, and eye geoms Jul 12, 2019
@mjskay mjskay added this to the Next release milestone Jul 14, 2019
mjskay added a commit that referenced this issue Jul 24, 2019
mjskay added a commit that referenced this issue Jul 24, 2019
mjskay added a commit that referenced this issue Jul 28, 2019
@mjskay mjskay changed the title Meta-geom: Make a single stat_density_interval / geom_density_interval from the hacked violins/densities, pointinterval, and eye geoms Meta-geom: Make a single stat_slabinterval / geom_slabinterval from the hacked violins/densities, pointinterval, and eye geoms Jul 30, 2019
@mjskay
Copy link
Owner Author

mjskay commented Aug 4, 2019

Done for now --- everything else relevant has been split into other issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant