Opened 9 years ago
Closed 9 years ago
#12836 closed defect (fixed)
arrows are shortened too much
| Reported by: | jason | Owned by: | jason, was |
|---|---|---|---|
| Priority: | major | Milestone: | sage-5.1 |
| Component: | graphics | Keywords: | |
| Cc: | kcrisman | Merged in: | sage-5.1.beta1 |
| Authors: | Jason Grout | Reviewers: | Michael Orlitzky |
| Report Upstream: | N/A | Work issues: | |
| Branch: | Commit: | ||
| Dependencies: | Stopgaps: |
Status badges
Description
Apparently 3 years ago, when I made an update patch for matplotlib, I put in some default shortening of arrows. Matplotlib already shortens its arrows to account for line width, so with my shortening and matplotlib's shortening, arrows were always way shorter than they were intended. This patch removes Sage's default shortening of arrows.
Attachments (4)
Change History (9)
Changed 9 years ago by
comment:1 Changed 9 years ago by
- Cc kcrisman added
- Status changed from new to needs_review
Changed 9 years ago by
Changed 9 years ago by
comment:2 Changed 9 years ago by
- Reviewers set to Michael Orlitzky
This change looks fine. The before/after.png show an arrow from the origin to (1,1) before and after the patch. I added a doctest that creates two arrows with different line widths, and checks that their shrinkA and shrinkB values are the same.
There's a chance I'm doing something dumb in the doctest, but positive review otherwise.
comment:3 Changed 9 years ago by
- Status changed from needs_review to positive_review
Your doctest patch looks good.
comment:4 Changed 9 years ago by
- Milestone changed from sage-5.0 to sage-5.1
comment:5 Changed 9 years ago by
- Merged in set to sage-5.1.beta1
- Resolution set to fixed
- Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
Needless to say, since it's a graphics change, people better look at it so people don't get terribly offended that there arrows look a little better :).