I'm a bit confused by this taking the data row-wise.
Theplot command accepts the data in columns, i.e. forplot(x,y) you need to have the first dimension ofy to match the length ofx.
Also for creating multiple histograms, the data for each histogram is in a single column of the array.
Finally, pandas bar plot function also uses the data column-wise.

So for consistency I would suggest to do the same here as well.

I did it the way it is because I think most users will have their data for each bar in an series of arrays and would like to just put them into one bar call. The post referenced in the feature request (https://stackoverflow.com/questions/14270391/python-matplotlib-multiple-bars?utm_medium=organic&utm_source=google_rich_qa&utm_campaign=google_rich_qa) does it that way so thats what I went with. I can change it if that's what you want but I'd like to be sure that others agree with you about it first.

I'd need to double check, but I somewhat suspect you broke the API forcolors etc as well. Or at the very least, you need to document the changes.

In terms of plotting just a single bar graph, nothing is changed. I could, however put in new documentation describing how optional parameters are treated by the multibar code.

I think there's a pretty bad issue right now of rows vs columns in this kind of data (see e.g.#8092) and would suggest we figure out the best way to handle that first (my opinion is in the linked issue, but mostly I just want things to be consistent :-)) before making the situation even more tangled...

A cultural standard is to organize data in tables. Those tables are read column by column.

Example (from the famous work by Jean Baptiste Perrin about Brownian Motion)

While of couse equally possible, I rarely see anyone creating such table by aranging the independent data horizontally.
Any modern computer tool I know of that is used for data storage or analysis uses tables in such way, including Database systems, Spreadsheet software, or any specialized tool I have been working with so far.

Example from Spreadsheet software:

More specifically, reading in data into numpy from such table with n rows and m columns, e.g. vialoadtxt, creates a numpy array of shape (n,m), so we get the data of the each table column along the first dimension of the numpy array. Equally pandas uses the concept of columns containing data from a series.

To be honest I'm at loss as to why one would suddenly use the transpose of that.

(I know that matplotlib already has this transposed concept e.g. in the case of asymmetrical errorbars, where data needs to be row-wise, but that is surely confusing.)

Ok. I made it take the data column-wise.

Data Structure

I think this is good. It is consistent with howplot andhist etc. handle 2D numpy arrays; and as outlined above this is in my optinion the correct way anyways. I do see the problem mentionned by@anntzer and discussed in#8092 for boxplots. I would argue that in the case of bar plots, there is no need to even support unequal length datasets (since you never acually want to leave out thelast bars, and hence need to usenp.nan for missing bars anyways, leading to nice 2D arrays.)

Documentation

A minor point (but not negligible) is the example:
There currently is abar chart example in the docs. This shows how to produce grouped bars (in the old fashionned way).
I see two options:

• Extend the existing example, i.e. Below it, write something like: "The above can now also be obtained by a single call tobar using a two-dimensional dataset for the heights with one column formen's data and another forwomen's data." And then show how to produce the exact same plot with this new feature.
• Link the old example to the new: "Note that grouped bar charts can now also be achieved through other means, see the (link:)grouped bar example". Then also link the new example to the old, e.g. "This now makes the creation of grouped bar chars as the one from the (link:) barchart example even easier."

(I think I slightly prefer the first option, simply because it keeps the number of different gallery examples low. But maybe it's also good to show an example with more than two groups?)

Finally, I think this grouping mostly makes sense for categorical data. Since matplotlib supports categorical data in the sense of strings supplied to plotting function, wouldn't it be nice to have this incoorporated in the example? So usingax.bar(labels, heights) instead ofax.bar(x, heights, label=labels).

While I understand the motivation, I have several concerns about this this.

If this goes in I suspect we will almost immediately get a request for this, but stacked. The obvious way to add this is to add some grouped-only relevant kwargs which is something we diffidently want to avoid.

I am also concerned about the change in return type based on the type of the input.

Theisinstance checks are troubling, we try very hard to avoid those. Is this always the broadcast semantic that we want? I think this will lead to a case where you can specify height column-wise, but row-wise for everyhing else.

How does this play with datastructures that are easy to get out of pandas dataframes?

I think this may be better as a new top-level function to avoid the variable return types, easily support the stacked case, and to (without concern for back compatibility) deal with the broadcasting of all the parameters issues.

How does this play with datastructures that are easy to get out of pandas dataframes?

With the function now taking the data in columns, this plays very well with pandas. As stated above this is one of the motivations to use columns instead of rows. Ifdf is a dataframe,

plt.bar(df.index, df.values)

It seems though that it does not allow for thedata argument to be provided. I guess this could be added, and take array-like arguments, likeplt.bar("x", ["y1", "y2"], colors=["colorcolumn1","colorcolumn2"], data=df); is that what is meant here?

I cannot say much about the other points - probably they are all valid.

If you guys could give me a checklist with what you want me to do I would be happy to oblige.

We talked about this on the weekly call and the consensus is that this is a very useful functionality, but the front end complexity people are going to want is quite large and going to rapidly grow (give a mouse a cookie....) and adding a large branch that recursively calls back into it's self is too much complexity on the implementation side.

The route we would like to see this take is to create a new function (multi_bar or something, (I'm really bad at names)) that is only for multi bar (both stacked and adjacent). As mentioned above this gets around almost all of the back-compatibility issues and shoe-horning the multi-bar API into the existing bar API. A good measure if the API is flexible enough is if we can replace the multi-dataset plotting inhist with a call to this function.

What kwargs frombar do we want to carry over?bottom andalign seem like candidates of things to drop, what else can go?

What are the desired semantics on:

• the data shape (I think this one is pretty pinned down as column-wise)
• how kwargs/style are broadcast. Setting one style per column makes sense, but would you ever want to broadcast by row? If so how would you differentiate user intent (particular for square imputs!)
• how to handle thedata kwarg through everything? (I like@ImportanceOfBeingErnest 's suggestion above)

Is it worth making a newArtist class for this so that users can change from stacked to adjacent interactively?

I think the next step is to write out the docstring for what this function!

Still thinking of working on this one@UTSC-SISYPHUS ?

Sorry, I had exams. I will still work on it. I'll get to it this weekend. I could do with a bit of information on thedataframe@ImportanceOfBeingErnest mentioned for the puprpose of supporting thedata kwarg though.

Added documentation for new function. Please give me feedback.

It looks like this currently only consists of documentation semantics. The code is simplypass.
Not sure what the status is here.

I had this working in another function but changes were requested. I'm asking if I should write the code for the documentation as it is now or if I should make some more changes first.

Since this Pull Request has not been updated in 60 days, it has been marked "inactive." This does not mean that it will be closed, though it may be moved to a "Draft" state. This helps maintainers prioritize their reviewing efforts. You can pick the PR back up anytime - please ping us if you need a review or guidance to move the PR forward! If you do not plan on continuing the work, please let us know so that we can either find someone to take the PR over, or close it.

Superseded by#24313. Please move further discussion there.