Skip to content

density_scatter_plotly add kwarg facet_col: str | None = None #193

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 10 commits into from
Aug 17, 2024
Merged

density_scatter_plotly add kwarg facet_col: str | None = None #193

merged 10 commits into from
Aug 17, 2024

Conversation

janosh
Copy link
Owner

@janosh janosh commented Aug 8, 2024 •
edited
Loading

minimal example

import pandas as pd
import plotly.express as px

df_tips = px.data.tips().rename(columns={"total_bill": "X", "tip": "Y", "size": "Z"})
fig = density_scatter_plotly(df_tips, x="X", y="Y", size="Z", facet_col="smoker")
fig.show()

density-scatter-plotly

@janosh janosh added enhancement Improvement to existing features/functionality plotly Concerning plotly-powered functions scatter Scatter plots labels Aug 8, 2024
janosh added 4 commits August 16, 2024 12:45
@janosh
ruff auto fixes
25df310
@janosh
support faceted plotly figs in pmv.utils.annotate
bb9d289
@janosh
add test_annotate_faceted_plotly
8a0ed3e 
test_annotate_faceted_plotly_with_empty_string
test_annotate_faceted_plotly_with_single_string
test_annotate_non_faceted_plotly_with_list
test_annotate_kwargs
@janosh
annotate_metrics support plotly facet figs
1e4a320
janosh added 2 commits August 17, 2024 19:50
@janosh
add_identity_line support plotly facet figs
cc2e369
@janosh
more add_best_fit_line tests
f358f78 
test_add_best_fit_line_custom_line_kwargs
test_add_best_fit_line_trace_idx
test_add_best_fit_line_faceted_plot
test_add_best_fit_line_custom_xs_ys
@janosh janosh enabled auto-merge (squash) August 17, 2024 18:07
@janosh janosh merged commit a1c02bb into main Aug 17, 2024
7 checks passed
@janosh janosh deleted the density-scatter-plotly-facet branch August 17, 2024 18:19
@janosh janosh mentioned this pull request Aug 18, 2024
Merged
janosh added a commit that referenced this pull request Mar 28, 2025
@janosh
density_scatter_plotly add kwarg facet_col: str | None = None (#193)
35d1c4b 
* density_scatter_plotly add kwarg facet_col: str | None = None

* Key enum add more DFT functionals

* ruff auto fixes

* support faceted plotly figs in pmv.utils.annotate

* add test_annotate_faceted_plotly
test_annotate_faceted_plotly_with_empty_string
test_annotate_faceted_plotly_with_single_string
test_annotate_non_faceted_plotly_with_list
test_annotate_kwargs

* annotate_metrics support plotly facet figs

* add_identity_line support plotly facet figs

* more add_best_fit_line tests

test_add_best_fit_line_custom_line_kwargs
test_add_best_fit_line_trace_idx
test_add_best_fit_line_faceted_plot
test_add_best_fit_line_custom_xs_ys

* add more tests for density_scatter_plotly
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement Improvement to existing features/functionality plotly Concerning plotly-powered functions scatter Scatter plots
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@janosh