Leave out barh from the basic plot types.
#23631
Conversation
`bar` / `barh` was the only plot type mentioning two functions, but the plot only showed the vertial one anyway. IHMO we can leave out `barh`. It's mentionend in the "See also" section of `bar`. (But I'm open to leaving a comment in the plot type description if desired - I mainly want to remove it from the title.)
I think this PR can be merged w/o that (feel free to self merge if you agree), but that it either might be a good idea to add notes/see alsos/etc on the plot_types plots when there's a very highly related functionality - like the barh or stacked bars or streamgraphs or it risks making the examples cluttered. |
|
It used to be the same as the cheatsheet: ======================
bar[h](x, height, ...)
====================== |
|
@story645 Thanks for the comment. I‘ll merge as is for now. I fundamentally agree that we could group together related functionality. But that goes in a slightly different direction than the current plot types, which are 1) actually function and not type based (*) and 2) intentionally are limited to a few selected types. So I leave that discussion for later. @jklymak Thanks for the reminder. I think I’ve discussed it before that IMHO (*) E.g. There’s overlap between plot with markers only and scatter; we have a few step-like functions (step, stairs, maybe hist); the mentioned stackedbars isn’t even a single function, but an algorithmic recipe (even more complicated are grouped bars). |
bar/barhwas the only plot type mentioning twofunctions, but the plot only showed the vertial one
anyway.
IHMO we can leave out
barh. It's mentionend in the"See also" section of
bar. (But I'm open to leavinga comment in the plot type description if desired -
I mainly want to remove it from the title, because it's
the only one spanning two lines:
)