Skip to content
New issue

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

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

Already on GitHub? # to your account

Adding plotting guides #2025

Merged
merged 15 commits into from
May 6, 2022
Merged

Conversation

symeneses
Copy link
Contributor

Description

Adds plotting guide per backend. It solves #1488

Checklist

Copy link
Member

@OriolAbril OriolAbril 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 so far, thanks! I'll continue reviewing as soon as possible

Copy link
Member

@OriolAbril OriolAbril left a comment

Choose a reason for hiding this comment

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

Everything looks great, reviewed already stylistic choices and typos

@symeneses symeneses force-pushed the doc/plotting-guides branch from 2bc36b7 to 11d1bde Compare May 5, 2022 13:41
@symeneses
Copy link
Contributor Author

Everything looks great, reviewed already stylistic choices and typos

Looks like all the conversations are solved now.

@OriolAbril OriolAbril merged commit 4364014 into arviz-devs:main May 6, 2022
@OriolAbril
Copy link
Member

Thanks!

@symeneses symeneses deleted the doc/plotting-guides branch May 9, 2022 09:10
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants