Skip to content

DOC: add "See Also: draw_idle" reference to pyplot.draw #24584

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 1 commit into from
Dec 5, 2022
Merged

DOC: add "See Also: draw_idle" reference to pyplot.draw #24584

merged 1 commit into from
Dec 5, 2022

Conversation

hukaidong
Copy link
Contributor

PR Summary

This is a simple solution following the issue #24578. Adding a reference draw_idle may reduce confusion to audiences who is visiting this page.

PR Checklist

Documentation and Tests

  • Has pytest style unit tests (and pytest passes)
  • Documentation is sphinx and numpydoc compliant (the docs should build without error).
  • New plotting related features are documented with examples.

Release Notes

  • New features are marked with a .. versionadded:: directive in the docstring and documented in doc/users/next_whats_new/
  • API changes are marked with a .. versionchanged:: directive in the docstring and documented in doc/api/next_api_changes/
  • Release notes conform with instructions in next_whats_new/README.rst or next_api_changes/README.rst

@tacaswell tacaswell modified the milestones: v3.6-doc, v3.6.3 Dec 2, 2022
@tacaswell tacaswell closed this Dec 2, 2022
@tacaswell tacaswell reopened this Dec 2, 2022
@tacaswell
Copy link
Member

Not sure what was wrong with circle, "powercycled" to see if that fixed it.

@tacaswell
Copy link
Member

I am still confused about why circle is failing.

@hukaidong
Copy link
Contributor Author

hukaidong commented Dec 4, 2022

I am still confused about why circle is failing.

You mean the circleCI pipeline? I think it fails in every pull request but don't know why. Here's what they said

Could not find a usable config.yml, you may have revoked the CircleCI OAuth app.
Please sign out of CircleCI and log back in with your VCS before triggering a new pipeline.

So it looks like some account configuration issue

@tacaswell
Copy link
Member

Circle is running on other PRs, would you be willing to rebase your PR onto current main (or I can do it for you, but do not want to force-push to your branch with out permission)?

@hukaidong
Copy link
Contributor Author

Circle is running on other PRs, would you be willing to rebase your PR onto current main (or I can do it for you, but do not want to force-push to your branch with out permission)?

rebased to latest commit but the problem seems consist

@hukaidong
Copy link
Contributor Author

Circle is running on other PRs, would you be willing to rebase your PR onto current main (or I can do it for you, but do not want to force-push to your branch with out permission)?

BTW you are welcomed if you want to do more test and force push on this branch. That will be much more easier

@tacaswell
Copy link
Member

If circle on #24622 passes I will merge this PR.

@tacaswell
Copy link
Member

See https://gitter.im/matplotlib/matplotlib?at=638d07ac4abb3f4684a40439 for discussion debugging this.

Our theory is that there is something confused on the circleCI side with @hukaidong 's account which is the source of the problem.

@tacaswell tacaswell merged commit aa1c538 into matplotlib:main Dec 5, 2022
@tacaswell
Copy link
Member

CI passed on #24622 so I'm going to merge this PR and close mine.

meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Dec 5, 2022
tacaswell added a commit that referenced this pull request Dec 5, 2022
…584-on-v3.6.x

Backport PR #24584 on branch v3.6.x (DOC: add "See Also: draw_idle" reference to pyplot.draw)
@hukaidong hukaidong deleted the draw_doc branch December 5, 2022 20:05
@ksunden ksunden mentioned this pull request Feb 20, 2023
6 tasks
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.

2 participants