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

docs(how-to): add graphviz section #8513

Merged
merged 1 commit into from
Mar 1, 2024
Merged

Conversation

cpcloud
Copy link
Member

@cpcloud cpcloud commented Mar 1, 2024

Add graphviz how-to.

@cpcloud cpcloud added this to the 9.0 milestone Mar 1, 2024
@cpcloud cpcloud added the docs Documentation related issues or PRs label Mar 1, 2024
@cpcloud cpcloud added the docs-preview Add this label to trigger a docs preview label Mar 1, 2024
@ibis-docs-bot ibis-docs-bot bot removed the docs-preview Add this label to trigger a docs preview label Mar 1, 2024
@ibis-docs-bot
Copy link

ibis-docs-bot bot commented Mar 1, 2024

@cpcloud cpcloud force-pushed the viz-docs branch 3 times, most recently from 57b4d07 to 26e223a Compare March 1, 2024 13:57
@lostmygithubaccount lostmygithubaccount added the docs-preview Add this label to trigger a docs preview label Mar 1, 2024
@ibis-docs-bot ibis-docs-bot bot removed the docs-preview Add this label to trigger a docs preview label Mar 1, 2024
@lostmygithubaccount
Copy link
Member

overall comment -- I understand this why this is in "Visualization", but should it instead be in "Analytics"? I imagine most people would use this to see their lineage and understand the operations they are running

perhaps the "Visualization" section should be renamed to "Plotting"? and this moved to "Analytics"?

@cpcloud
Copy link
Member Author

cpcloud commented Mar 1, 2024

I think this will be harder to find in Analytics. I don't think we can anticipate all reasons why someone might want to visualize an Ibis expression as a graph.

It may be for lineage or other reasons.

IMO it's sufficient to provide documentation for the specific question of "How do I visualize the expression as a graph?", which is what I am trying to address with this PR.

@ibis-docs-bot
Copy link

ibis-docs-bot bot commented Mar 1, 2024

@cpcloud cpcloud added the docs-preview Add this label to trigger a docs preview label Mar 1, 2024
@ibis-docs-bot ibis-docs-bot bot removed the docs-preview Add this label to trigger a docs preview label Mar 1, 2024
Copy link
Member

@lostmygithubaccount lostmygithubaccount left a comment

Choose a reason for hiding this comment

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

awesome! one minor comment

docs/how-to/visualization/graphs.qmd Outdated Show resolved Hide resolved
@ibis-docs-bot
Copy link

ibis-docs-bot bot commented Mar 1, 2024

@cpcloud cpcloud enabled auto-merge (squash) March 1, 2024 15:50
@cpcloud cpcloud merged commit f1bb060 into ibis-project:main Mar 1, 2024
15 checks passed
@cpcloud cpcloud deleted the viz-docs branch March 1, 2024 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation related issues or PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants