Skip to content

Conversation

@sayalaruano
Copy link
Collaborator

@sayalaruano sayalaruano commented Sep 15, 2025

Description

Add the violin plot to VueCore.

Tasks Checklist

  • Create Pydantic schema in the vuecore/schemas folder. It should be aligned with the plotly API
  • Create a script with a build function in the vuecore/engines/plotly folder
  • Update theming.py script in the vuecore/engines/plotly folder
  • Add the new plot in the PlotType StrEnum of the vuecore/constants.py script, and register the new builder in the __init__.py script of the vuecore/engines/plotly folder
  • Create a script with the user-facing function in the vuecore/plots folder. It gathers the Pydantic schema, builder function, and saves the plot
  • Create an api example jupyter notebook in the docs/api_examples folder
  • Use jupytext to sync the Jupyter notebook with a Python script
  • Update index.md file in the docs folder with the new example
  • Create test script in the /test folder with the code from the example

sayalaruano and others added 13 commits September 15, 2025 16:01
…unction for the violin plot, and update box plot api examples names docstring
…meter from violin config as expected by go.Figure in the apply_violin_theme function
…example for box plot to also include the violin plot, and sync it with a python script
…ile and rename test_scatterplot.py file to follow the naming standard of the other scripts
@sayalaruano sayalaruano requested a review from enryH September 17, 2025 09:03
Copy link
Collaborator

@enryH enryH left a comment

Choose a reason for hiding this comment

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

Looks great. I will in my PR #40 merge this in and try to add the links in the api examples to the create_violion_plot function in the notebook and in the docstrings have the Schema Objects linked as well.
Somthing like this:

:func:`~vuecore.plots.basic.scatter.create_scatter_plot`

@sayalaruano sayalaruano merged commit 6e19f17 into main Sep 17, 2025
7 checks passed
@sayalaruano sayalaruano deleted the add-violin-plot branch September 17, 2025 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants