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

fix the docs build and add an easy example for subplots using seaborn #242

Merged
merged 3 commits into from
Jun 12, 2023
Merged

fix the docs build and add an easy example for subplots using seaborn #242

merged 3 commits into from
Jun 12, 2023

Conversation

jkittner
Copy link
Contributor

@jkittner jkittner commented May 18, 2023

  • also add a copy button to all code block for easier usage of the docs

I think there were some issues deploying the docs in the past - they don't seem to be up to date with what's in the repo. I couldn't test the deploy workflow, but building works fine for me now (locally and in gha).

The docs used to have a subplots example, which unfortunately was broken and also quite complex. I think the most straight forward way to use this with subplots is seaborn.

In general I think subplots are commonly used and having an easy way of creating them in the docs is nice, since windroses in general lack the temporal component.

This is the output of the example:

t

jkittner and others added 2 commits June 12, 2023 11:58
- also add a copy button to all code block for easier usage of the docs
docs/conf.py Show resolved Hide resolved
docs/index.rst Outdated Show resolved Hide resolved
@ocefpaf ocefpaf merged commit a6b8de5 into python-windrose:main Jun 12, 2023
15 checks passed
@ocefpaf
Copy link
Collaborator

ocefpaf commented Jun 12, 2023

Super cool! Thanks for the PR @jkittner!

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.

Add subplots script link to documentation Add margin, spacing, and a better API for subplots
2 participants