Skip to content

Merge recent docs updates #4589

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

Merged
merged 23 commits into from
Apr 30, 2024
Merged

Merge recent docs updates #4589

merged 23 commits into from
Apr 30, 2024

Conversation

LiamConnors
Copy link
Member

@LiamConnors LiamConnors commented Apr 25, 2024

Merge recent docs updates into master in preparation for the release
https://github.com/plotly/plotly.py/blob/master/release.md#update-documentation-site

rl-utility-man and others added 23 commits March 13, 2024 23:06
renamed df2 to df
1)  added an example that formats the elements of custom_data in plotly express
2)  fixed a bug in the "advanced hovertemplate" example which caused it to display the square root of population.   Modified that example to follow the main dataframe is called "df" convention.  Added a country name field to that example and comments about the usage of key parameters.
3)  I did not remove the "Adding other data to the hover with customdata and a hovertemplate" example, but I believe it is now redundant and that the revised ### Advanced Hover Template example is more compliant with expectations like using meaningful data over random data.  If you removed it, it would make sense to rename "Advanced Hover Template" to include the phrase:  "Adding other data to the hover with customdata and a hovertemplate"
proposed addition of a df.agg stacked go.bar example
Code block under the subtitle 'Stacked Bar Chart From Aggregating a DataFrame' wasn't rendering because language declaration for the block was missing.
fix: Code block not rendering
Update hover-text-and-formatting.md
@LiamConnors LiamConnors requested a review from emilykl April 25, 2024 17:48
@LiamConnors LiamConnors marked this pull request as ready for review April 25, 2024 17:48
@LiamConnors LiamConnors requested a review from archmoj April 30, 2024 18:27
Copy link
Contributor

@archmoj archmoj left a comment

Choose a reason for hiding this comment

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

💃

@LiamConnors LiamConnors merged commit 9a158eb into master Apr 30, 2024
5 checks passed
@LiamConnors LiamConnors deleted the recent-docs-updates branch April 30, 2024 20:22
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.

4 participants