Violinplot-Matlab icon indicating copy to clipboard operation
Violinplot-Matlab copied to clipboard

Display Mean

Open RoyiAvital opened this issue 9 years ago • 15 comments

This is an amazing visualization tool!

Could you add an option to display the Mean of the samples in addition to the Median? Just adding another marker.

Thank You.

RoyiAvital avatar Oct 12 '16 08:10 RoyiAvital

Hi,

I see you update the code frequently.

Could you use Releases for mile stones with release notes?

Thank You.

RoyiAvital avatar Jul 01 '17 08:07 RoyiAvital

Could you add an option to display the Mean of the samples in addition to the Median?

This would go against the philosophy of a violin plot. How would you propose to show the mean? Should it include outliers? I'm genuinely curious.

I see you update the code frequently. Could you use Releases for mile stones with release notes?

So far, I pushed ten commits, most of which were bug fixes. I can certainly tag an arbitrary version, though. Would that be satisfactory?

bastibe avatar Jul 03 '17 09:07 bastibe

@bastibe, Interesting question whether it should include outliers or not. The easy option is to have a parameter to set how to calculate it.

Think of it as an option to display many histograms of distributions. It would be great to be able to compare, in one plot, as many features of the distribution as possible. Hence I'd say the mean, by default, should be calculated using all samples.

Regarding the commits and release version. Well it would be nice that people will be able to update their function. Not everybody are doing it using GIT, some just download the ZIP. So I guess the best option is to add a text file of Release Notes and write there any update made.

Thank You.

RoyiAvital avatar Jul 03 '17 10:07 RoyiAvital

I implemented a mean indicator. Could you check if this works for you?

bastibe avatar Jul 03 '17 14:07 bastibe

@bastibe ,

You really add release notes or versioning to README.md or one of the MATLAB files. Currently someone who has it already can never know what version he has.

Moreover, in the documentation in violinplot suggests that cats can not be just regular numeric values while it can (At least the impression is like that). So I would add a case where mData is m x n numeric matrix and vCats is n x 1 or 1 x n numeric vector. Then a violin is made for each column i of mData for the vCats(i) numeric category.

RoyiAvital avatar Jun 22 '18 20:06 RoyiAvital

Would you want to take a stab at that? I would be grateful if you created a pull request with better README!

bastibe avatar Jun 23 '18 08:06 bastibe

Hi, I suggest that you enable Wiki for the repository and then I will be able to add information there.

RoyiAvital avatar Jun 23 '18 14:06 RoyiAvital

To be honest, I'd prefer this information in the README. But I'll enable the Wiki if you're not comfortable with editing the README.

bastibe avatar Jun 24 '18 10:06 bastibe

I don't know the whole Pull Process thing. It is much easier for me to contribute in Wiki.

I think documentation is better done by Wiki. Yet the version must be added in README.md.

RoyiAvital avatar Jun 24 '18 11:06 RoyiAvital

I don't know the whole Pull Process thing.

There should be an edit button in Github when you're looking at the README file (not the project overview, you have to click on the README file once). This should open the README for editing, and package up the result as a Pull Request automatically.

Still, if you prefer the Wiki, I'll pull it into the README afterwards.

bastibe avatar Jun 24 '18 11:06 bastibe

@bastibe ,

The Wiki can be a place for users to write notes and stuff about the code and the usage. But release notes is something only you can.

I like documenting release notes within the function itself. For instance have a look at https://github.com/RoyiAvital/Projects/blob/master/Optimization/LsL1SolversAnalysis/SolveLsL1Irls.m.

RoyiAvital avatar Jun 24 '18 13:06 RoyiAvital

At the moment, the Wiki contains only Github's boiler plate. Feel free to add or remove as you see fit.

bastibe avatar Jun 26 '18 12:06 bastibe

@bastibe , I guess I'm not describing what I mean.

Leave alone the Wiki. It has nothing to do with versioning.

What I think is needed is to add information about the version in one of the 3 files in the toolbox (README.md, Violin.m and violinplot.m). Something to let the user be able to say on what version he works compared to the latest on GIT.

RoyiAvital avatar Jun 26 '18 16:06 RoyiAvital

Let's get back on topic. Are you satisfied with the implementation of the mean indicator?

If you want to discuss further topics (adding documentation, adding a changelog, adding versions), please open a new issue.

bastibe avatar Jun 27 '18 07:06 bastibe

Good point. We should probably use the same grey tone as all the other lines. Would you like to contribute the change as a pull request?

bastibe avatar Oct 27 '22 08:10 bastibe