-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
[Term Entry] Sankey #6268
[Term Entry] Sankey #6268
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I ran the code snippet locally and the output remains the same so this works :) I only have few comments for you to work on let's fix them and move this PR for second review.
Hi @dakshdeepHERE, I should have resolved all the problems. Let me know if there are any others. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the changes this looks good for second review
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@alessandrocapialbi The entry looks good to be merged 🚀
Thank you for contributing! 😄
👋 @alessandrocapialbi 🎉 Your contribution(s) can be seen here: https://www.codecademy.com/resources/docs/plotly/graph-objects/sankey Please note it may take a little while for changes to become visible. |
Description
I have created a new term entry titled sankey.md adding a new directory named 'sankey'' under the terms directory in graph_objects concepts. This entry provides a comprehensive overview of the .Sankey() method in Plotly, which is used to generate Sankey diagrams.
Issue Solved
Closes #6251
Type of Change
Checklist
main
branch.Issues Solved
section.