Skip to content

Conversation

anntzer
Copy link
Contributor

@anntzer anntzer commented Oct 4, 2017

The profiled values are way outdated (they were computed back in 2008,
so even the rendering strategy has changed), and if we want to have some
speed benchmarks they should go to some more prominent place than a
comment at the top of an example.

(see https://matplotlib.org/devdocs/gallery/lines_bars_and_markers/scatter_profile.html#sphx-glr-gallery-lines-bars-and-markers-scatter-profile-py)

PR Summary

PR Checklist

  • Has Pytest style unit tests
  • Code is PEP 8 compliant
  • New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant
  • Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
  • Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

The profiled values are way outdated (they were computed back in 2008,
so even the rendering strategy has changed), and if we want to have some
speed benchmarks they should go to some more prominent place than a
comment at the top of an example.
@tacaswell tacaswell merged commit 922156f into matplotlib:master Oct 4, 2017
tacaswell added a commit that referenced this pull request Oct 4, 2017
DOC: Remove scatter_profile example.
@tacaswell
Copy link
Member

backported to v2.1.0-doc as 5f1762e

@QuLogic QuLogic added this to the 2.1.0-docs milestone Oct 4, 2017
@anntzer anntzer deleted the remove-scatter_profile branch October 4, 2017 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants