Switch whisker plots to violin plots

Nishi46 commented 3 months ago

switched whisker plot to violin plot in multi_classifier.py as stated in Issue #36

CLAassistant commented 3 months ago

All committers have signed the CLA.

diehlbw commented 3 months ago

Thank you so much for taking on this update!

A few minor things/more places to update:

[x] Can you update your PR header, I think you referenced 37 instead of 36.
[x] Add a one-liner newsfragment file in changelog/ (I'm being intentionally vague here as this will help test out that the README works)
[ ] Update a couple other references of 'whisker', you got the code updates so this is just bookkeeping documentation and tests.
- [x] docs/reference/internals.rst is the most important, I expect CI will fail before that update
- [ ] *.ipynb mention it in markdown (in the template/, example/, and also in a few test expectation files)

epic-open-source / seismometer