Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Patch issue 6035 rebase #6504

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.

Sign up for GitHub

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 12 commits into from
May 30, 2016
Merged

Patch issue 6035 rebase #6504

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
d1fddc7
First draft to correct issue #6035 and to write relevant unit tests. …
afvincent Feb 23, 2016
f0eda1f
Achieve DPI-independent arrow heads in a cleaner way
afvincent Feb 23, 2016
20b46db
Add a new unit test to check that a raw FancyArrowPatch is DPI-indepe…
afvincent Feb 23, 2016
95d1fb0
Docstring fix
afvincent Feb 23, 2016
85cecf1
Removed whitespaces (for PEP8)
afvincent Feb 23, 2016
3088b35
Added 2nd whitespace before inline comment (PEP8)
afvincent Feb 23, 2016
6191dff
Should fix broken unit tests 'polar_axes' and 'polar_coords' in test_…
Feb 25, 2016
d3af67d
Docstring/comments fixes & PEP8.
afvincent Mar 29, 2016
d131fd8
Docstring fix
afvincent Apr 1, 2016
a3ed8af
Update of failing PNGs in test_axes
afvincent Apr 1, 2016
5b464e7
New rebase and update of wrong PNG baseline images
afvincent May 14, 2016
d6eee04
Make __prepare_fancyarrow_dpi_cor_test private to prevent nose from r…
jenshnielsen May 30, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Achieve DPI-independent arrow heads in a cleaner way
  • Loading branch information
@afvincent @jenshnielsen
afvincent authored and jenshnielsen committed May 30, 2016
commit f0eda1f4147b5984af9512daf9ff063b575da9fc
26 changes: 11 additions & 15 deletions lib/matplotlib/patches.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4056,23 +4056,19 @@ def __init__(self, posA=None, posB=None,

self._mutation_scale = mutation_scale
self._mutation_aspect = mutation_aspect
# Provide a ref. to scale '_mutation_scale' when setting '_dpi_cor'.
self._genuine_mutation_scale = mutation_scale


self.set_dpi_cor(dpi_cor)
#self._draw_in_display_coordinate = True

def set_dpi_cor(self, dpi_cor):
"""
dpi_cor is currently used for linewidth-related things and
shrink factor. Mutation scale *is* affected by this.
shrink factor. Mutation scale is not affected by this.
"""

self._dpi_cor = dpi_cor
self.stale = True
# Scale `_mutation_scale` properly with the new DPI (see issue #6035).
self._mutation_scale = self._genuine_mutation_scale * dpi_cor


def get_dpi_cor(self):
"""
dpi_cor is currently used for linewidth-related things and
Expand Down Expand Up @@ -4233,10 +4229,10 @@ def get_path_in_displaycoord(self):
_path = self.get_transform().transform_path(self._path_original)

_path, fillable = self.get_arrowstyle()(_path,
self.get_mutation_scale(),
self.get_linewidth() * dpi_cor,
self.get_mutation_aspect()
)
self.get_mutation_scale() * dpi_cor,
self.get_linewidth() * dpi_cor,
self.get_mutation_aspect()
)

#if not fillable:
# self._fill = False
Expand Down Expand Up @@ -4549,10 +4545,10 @@ def get_path_in_displaycoord(self):
)

_path, fillable = self.get_arrowstyle()(_path,
self.get_mutation_scale(),
self.get_linewidth() * dpi_cor,
self.get_mutation_aspect()
)
self.get_mutation_scale() * dpi_cor,
self.get_linewidth() * dpi_cor,
self.get_mutation_aspect()
)

return _path, fillable

Expand Down
1 change: 0 additions & 1 deletion lib/matplotlib/text.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2147,7 +2147,6 @@ def _update_position_xytext(self, renderer, xy_pixel):

d = self.arrowprops.copy()
ms = d.pop("mutation_scale", self.get_size())
ms = renderer.points_to_pixels(ms)
self.arrow_patch.set_mutation_scale(ms)

if "arrowstyle" not in d:
Expand Down