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

Merge changes from doc-prod to master #4816

Merged
merged 32 commits into from
Oct 22, 2024
Merged

Conversation

LiamConnors
Copy link
Member

No description provided.

SeanMcCarren and others added 30 commits May 25, 2022 09:48
This is only a minor mistake, but slightly misleading because for a moment I thought this example showed how to draw only some but not all edges.
proposed by empet on the forum
AUC for PR curve needs precision and recall as their input. fpr and tpr would be appropriate input for ROC curve.
- There is no need for passing dummy variable `y` when stats have already been computed
- `upperfence` values don't match the resulting figure.
- It could be simplified even further by getting rid of the extra call to `add_trace` but I've kept it to follow the documentation styling.
- It might be worth it to separate the example in two cases: (i) with only `q1`, `median`, `q3` provided, (ii) with all values provided.
fixes #3274

The list often serves as reference to look for modebar options and v1hovermode was missing.
Fix "Update Button" example in Python custom buttons doc page
Align an annotation with the center of a donut pie
make moving-point-on-a-curve faster
Add v1hovermode to the list of all the modebar options
Remove errroneous comma in SVG path
…e continent

There is a bug here that causes the hovertext to show one line as %{text} rather than showing the continent name.  I have fixed it the way the comments imply that the example intended.  either we should add a text parameter as I've done here, or the example could remove the %{text} from the hovertext and from the comments.
fixing a bug that causes the hovertext to show %{text} rather than th…
@LiamConnors LiamConnors requested a review from archmoj October 22, 2024 13:55
@gvwilson gvwilson added feature something new P1 needed for current cycle fix fixes something broken labels Oct 22, 2024
@gvwilson gvwilson self-requested a review October 22, 2024 14:03
@archmoj
Copy link
Contributor

archmoj commented Oct 22, 2024

@LiamConnors There are now some conflicts here.
Are you comfortable resolving them?

@LiamConnors LiamConnors merged commit b64f288 into master Oct 22, 2024
4 of 5 checks passed
@LiamConnors LiamConnors deleted the merge-doc-prod-to-master branch October 22, 2024 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature something new fix fixes something broken P1 needed for current cycle
Projects
None yet
Development

Successfully merging this pull request may close these issues.