Opened 10 years ago
Last modified 5 years ago
#7808 needs_info defect
Plots seem to be shifted up and to the left by one pixel or so
Reported by: | jason | Owned by: | was |
---|---|---|---|
Priority: | major | Milestone: | sage-6.4 |
Component: | graphics | Keywords: | |
Cc: | Merged in: | ||
Authors: | Jason Grout | Reviewers: | |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
There was a thread on sage-devel where it seemed like a number of people felt plots (or axes!) were shifted by about one pixel or so. Here's one manifestation:
region_plot([x^2+y^2<=1, x<=y], (x,-2,2), (y,-2,2),plot_points=400).show(aspect_ratio=1)
There are a lot of other examples in a thread on one of the sage mailing lists, but I can't find the thread.
Attachments (1)
Change History (21)
comment:1 Changed 9 years ago by
comment:2 Changed 9 years ago by
Here's an even better illustration. Try changing the dpi:
@interact def show_shift(dpi=(50,150,1)): # 50 dpi = shifted left # 70 dpi = shifted up # 80 dpi = seems perfect # 100 dpi = shifted up and left # 105 dpi = shifted up # 110 dpi = seems perfect bl = disk((0.0,0.0), 1, (pi, 3*pi/2), color='yellow') tr = disk((0.0,0.0), 1, (0, pi/2), color='yellow') tl = disk((0.0,0.0), 1, (pi/2, pi), color='black') br = disk((0.0,0.0), 1, (3*pi/2, 2*pi), color='black') P = tl+tr+bl+br P.show(figsize=[10,10],dpi=dpi,aspect_ratio=1,xmin=-2,xmax=2,ymin=-2,ymax=2)
comment:3 Changed 9 years ago by
One more iteration, this time with the option to see the frame:
@interact def show_shift(dpi=(50,150,1),frame=False): # 50 dpi = shifted left # 70 dpi = shifted up # 80 dpi = seems perfect # 100 dpi = shifted up and left # 105 dpi = shifted up # 110 dpi = seems perfect # different results if frame is checked bl = disk((0.0,0.0), 1, (pi, 3*pi/2), color='yellow') tr = disk((0.0,0.0), 1, (0, pi/2), color='yellow') tl = disk((0.0,0.0), 1, (pi/2, pi), color='black') br = disk((0.0,0.0), 1, (3*pi/2, 2*pi), color='black') P = tl+tr+bl+br P.show(figsize=[10,10],dpi=dpi,frame=frame,aspect_ratio=1,xmin=-2,xmax=2,ymin=-2,ymax=2)
comment:4 Changed 9 years ago by
Actually, the above examples might just be due to the line width (the line around the disk). Here is an example that again shows something really is wrong:
p=line([[-1,0],[1,0]])+line([[0,-1],[0,1]],color='red')+line([[-1,-1],[1,1]],color='yellow') p.show(figsize=20,dpi=300)
Zooming in very closely shows that the red and blue lines are perfect, but the yellow line is shifted up or to the left or both (note the yellow pixels in quadrant 2, but not in quadrant 4).
comment:5 Changed 9 years ago by
I posted a message to matplotlib-devel with what I think is a good example illustrating this in straight matplotlib: http://permalink.gmane.org/gmane.comp.python.matplotlib.devel/8664
Changed 9 years ago by
comment:6 Changed 9 years ago by
- Status changed from new to needs_info
I've attached a patch which is a workaround solving the problem. However, it causes other side effects (fuzzy lines, since now lines are not snapped to whole-pixels). You can uncomment the antialiased line in the patch to not get fuzzy lines, but then sometimes one axis will be one pixel wider than the other, which looks bad. I haven't tested how these options affect vector graphics output (e.g., pdf).
So we have three options:
- Leave things the way they are
- Turn off snapping, but deal with possible fuzzy lines
- Turn off snapping and turn off antialiasing, and deal with axes that sometimes don't match in width.
Pick your poison.
comment:7 Changed 9 years ago by
comment:8 Changed 9 years ago by
Instead of applying the patch above, you can also do:
- Install #9208 (to the scripts repository)
- If you've moved Sage from the build directory, then move it back to the original build location, delete SAGE_ROOT/local/lib/sage-current-location.txt, install #9210, start up and close down Sage, (and then you can move the sage directory back to wherever you had it in step 1).
- Install the new matplotlib spkg at #9202
Now just type this in a notebook cell:
import matplotlib matplotlib.rcParams['path.snap'] = False
Then the plots will go right through the origin. This is equivalent (better, actually), than the patch above, as it makes *all* horizontal/vertical lines not snap to the nearest pixel.
So now the debate is if we should make path.snap by default False (it is True by default in matplotlib).
comment:9 Changed 9 years ago by
The new matplotlib spkg with the path.snap parameter has been moved to #9221
comment:10 follow-up: ↓ 11 Changed 9 years ago by
I'm uncomfortable with the following comment in the patch
(but drawing another # horizontal/vertical line through the origin may not!)
But would that still be the case for the mpl upgrade piece?
Otherwise it seems that 2 is the best option. Axes that don't match is a similarly annoying thing to the moved-over pixel.
comment:11 in reply to: ↑ 10 Changed 9 years ago by
Replying to kcrisman:
I'm uncomfortable with the following comment in the patch
(but drawing another # horizontal/vertical line through the origin may not!)But would that still be the case for the mpl upgrade piece?
Setting path.snap to False makes all horizontal/vertical lines do the .set_snap(False) (not just the axes). So using the rcParam as above would make all horizontal and vertical lines consistent.
Otherwise it seems that 2 is the best option. Axes that don't match is a similarly annoying thing to the moved-over pixel.
In that case, all we have to do is set the matplotlib path.snap config parameter somewhere (either in our code, or provide a default matplotlibrc, or modify the default matplotlibrc in the spkg.
I played a little with Mathematica, and it seems like they have both crisp horizontal lines *and* perfect axes. I'm not quite sure what they are doing, though.
comment:12 Changed 9 years ago by
Well, anyway one would want to have pictures to vote on first :)
comment:13 Changed 9 years ago by
comment:14 follow-up: ↓ 15 Changed 9 years ago by
Is this now ready for review, with #9221 merged? Or do we still have to decide on one of 1-3?
comment:15 in reply to: ↑ 14 ; follow-up: ↓ 16 Changed 9 years ago by
Replying to kcrisman:
Is this now ready for review, with #9221 merged? Or do we still have to decide on one of 1-3?
I don't think the patch should be merged. Instead, a new patch that does this:
import matplotlib matplotlib.rcParams['path.snap'] = False
somewhere should be posted. I think a vote should also be taken on sage-devel with before and after pics.
comment:16 in reply to: ↑ 15 Changed 9 years ago by
I don't think the patch should be merged. Instead, a new patch that does this:
import matplotlib matplotlib.rcParams['path.snap'] = Falsesomewhere should be posted. I think a vote should also be taken on sage-devel with before and after pics.
Thanks for volunteering :) Thankfully, I've not even been taking home a computer that can compile Sage in less than one day home lately.
The reason I thought of this is a recent ask.sagemath.org post.
comment:17 Changed 6 years ago by
- Milestone changed from sage-5.11 to sage-5.12
comment:18 Changed 6 years ago by
- Milestone changed from sage-6.1 to sage-6.2
comment:19 Changed 5 years ago by
- Milestone changed from sage-6.2 to sage-6.3
comment:20 Changed 5 years ago by
- Milestone changed from sage-6.3 to sage-6.4
Here is another example which clearly shows something is wrong:
(from the disk doctests)