Skip to content
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.

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

Breaking: rename custom plotly template pymatviz_(black->dark) #123

Merged
merged 11 commits into from
Feb 11, 2024

Conversation

janosh
Copy link
Owner

@janosh janosh commented Feb 11, 2024

5e6717a breaking: rename custom plotly template pymatviz_black to pymatviz_dark
83cf9a3 add test_pkg_metadata
bb13a38 add keyword post_process: Callable[[str], str] to df_to_html_table()
b17d6be make file_path optional in df_to_html_table() and return HTML string
04089c5 test return value in test_df_to_html_table and no file written if file_path=None

don't set px.defaults.template = "pymatviz_white", pio.templates.default = "pymatviz_white" when importing pymatviz (probably unwanted)
copy plotly_white/dark before updating (no more modify in place)
@janosh janosh added plotly Concerning plotly-powered functions breaking Breaking changes api Application programming interface pandas Dataframe and Series related stuff labels Feb 11, 2024
janosh and others added 6 commits February 11, 2024 17:26
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/code/tests/test_readme.py", line 3, in <module>
    from pymatviz import ROOT
  File "/code/pymatviz/__init__.py", line 16, in <module>
    import matplotlib.pyplot as plt
ModuleNotFoundError: No module named 'matplotlib'
@janosh janosh merged commit 820a2d2 into main Feb 11, 2024
4 of 5 checks passed
@janosh janosh deleted the fix-pmv-plotly-templates branch February 11, 2024 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api Application programming interface breaking Breaking changes pandas Dataframe and Series related stuff plotly Concerning plotly-powered functions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant