-
-
Notifications
You must be signed in to change notification settings - Fork 544
HTML to ipynb update for scatter and line charts #1047
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
HTML to ipynb update for scatter and line charts #1047
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 was going to comment inline but I think it will be easier to write here (I put a couple in there).
I think it would be worthwhile consolidating all 4 of the Matplotlib docs into 2 docs that would align more with the native Plotly docs.
Thus,
https://plot.ly/matplotlib/lines-and-markers/
https://plot.ly/matplotlib/scatter/
https://plot.ly/matplotlib/plot/
https://plot.ly/matplotlib/line-and-scatter/
To,
Scatter Plots https://plot.ly/matplotlib/line-and-scatter/
Line Charts https://plot.ly/matplotlib/line-charts/
That way the URLs are the same and the Matplotlib docs are linked with the native Plotly.
Unless the Matplotlib URLs are like that for searching purposes @cldougl ? Though I don't think it would be.
@@ -0,0 +1,292 @@ | |||
--- | |||
permalink: matplotlib/line-and-scatter/ | |||
description: How to make line and scatter plots in matplotlib. Seven examples of basic and colored line and scatter plots. |
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'd remove this Seven examples of basic and colored line and scatter plots.
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.
👍
layout: user-guide | ||
language: matplotlib | ||
title: Matplotlib Line and Scatter Plots | Plotly | ||
display_as: chart_type |
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.
basic?
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.
oops yeah. supposed to be basic
We probably are trying to get results from 'matplotlib plot' with https://plot.ly/matplotlib/plot/ For now I would suggest keeping https://plot.ly/matplotlib/plot/ and then consolidating: https://plot.ly/matplotlib/lines-and-markers/ To: and adding redirects for the old urls: https://plot.ly/matplotlib/lines-and-markers/ --> https://plot.ly/matplotlib/line-charts/ how does that sound @bcdunbar @priyatharsan ? |
Agreed! sg 👍 |
Sg to me as well. |
@priyatharsan sweet 👍 Could you sweep through the
|
Thanks for making the changes. Same as mentioned in other PRs, update the reference section to point to the reference page https://plot.ly/python/reference/ |
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.
Thanks 💃
Update for scatter, line charts and line and scatter #802