Add Axes.violinplot datetime test by tanvincible · Pull Request #27485 · matplotlib/matplotlib · GitHub
Skip to content

Add Axes.violinplot datetime test - #27485

Closed
tanvincible wants to merge 0 commit into
matplotlib:mainfrom
tanvincible:mtanvi19-add-violinplot-datetime-test
Closed

Add Axes.violinplot datetime test#27485
tanvincible wants to merge 0 commit into
matplotlib:mainfrom
tanvincible:mtanvi19-add-violinplot-datetime-test

Conversation

@tanvincible

Copy link
Copy Markdown

PR summary

Add complementary test for axes.violinplot in Matplotlib
This is modeled for Axes.violinplot in #26864

Image of generated plot:

Figure 2023-12-09 174127

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for opening your first PR into Matplotlib!

If you have not heard from us in a week or so, please leave a new comment below and that should bring it to our attention. Most of our reviewers are volunteers and sometimes things fall through the cracks.

You can also join us on gitter for real-time discussion.

For details on testing, writing docs, and our review process, please see the developer guide

We strive to be a welcoming and open project. Please follow our Code of Conduct.

@jklymak

jklymak commented Dec 9, 2023

Copy link
Copy Markdown
Member

@ksunden

ksunden commented Dec 11, 2023

Copy link
Copy Markdown
Member

@QuLogic QuLogic left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is no longer a test, so cannot be merged as is (regardless of the other concerns above.)

@tanvincible
tanvincible deleted the mtanvi19-add-violinplot-datetime-test branch December 13, 2023 15:59
@melissawm melissawm moved this from Waiting for author to Done in First Time Contributors Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

5 participants