Opened 14 years ago
Closed 13 years ago
#2076 closed defect (fixed)
[with patch, positive review] Inconsistent coloring of plotted points
Reported by: | jmitchell | Owned by: | mhansen |
---|---|---|---|
Priority: | minor | Milestone: | sage-3.1.3 |
Component: | graphics | Keywords: | color hue point plot 3 |
Cc: | Merged in: | ||
Authors: | Reviewers: | ||
Report Upstream: | Work issues: | ||
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
When I call,
sage: point(((1,1), (2,2), (3,3)), rgbcolor=hue(1), pointsize=30)
I expect to get a plot of 3 red points, but (1,1) is plotted as dark red and the other two points are blue. So far I've only been able to recreate this issue when only three points are in the tuple. The rgbcolor value doesn't seem to affect the color of the three plotted points.
I hope to look at the code soon.
Attachments (1)
Change History (11)
comment:1 Changed 14 years ago by
- Milestone set to sage-2.10.2
comment:2 Changed 14 years ago by
comment:3 Changed 14 years ago by
I propose the following:
- We upgrade from
matplotlib-0.91.1
to the latest verison, whatever it is.
- If this doesn't fix the problem, report it to the matplotlib list.
- If that doesn't work, fix the problem !? Or program around it.
comment:4 Changed 13 years ago by
- Owner changed from was to mhansen
- Status changed from new to assigned
Changed 13 years ago by
comment:5 Changed 13 years ago by
- Summary changed from Inconsistent coloring of plotted points to [with patch, needs review] Inconsistent coloring of plotted points
This is actually a bug on our part according to the matplotlib 0.98 documentation for scatter.
comment:6 Changed 13 years ago by
I've read the matplotlib docs and this looks reasonable. I don't have a current 3.1.2 tree to test it, though; so half of a positive review (the other half comes from applying it and checking the result of the doctest :).
comment:7 Changed 13 years ago by
Jason,
since you now have a 3.1.2 can you do the final review on this? I can run doctests without a problem on 3.1.3.alpha1 to see if anything breaks.
Cheers,
Michael
comment:8 Changed 13 years ago by
The example now works and doctests in plot/*.py pass with the patch applied. So that's a full positive review, now.
comment:9 Changed 13 years ago by
- Summary changed from [with patch, needs review] Inconsistent coloring of plotted points to [with patch, positive review] Inconsistent coloring of plotted points
comment:10 Changed 13 years ago by
- Resolution set to fixed
- Status changed from assigned to closed
Merged in Sage 3.1.3.alpha1
Amazingly, this is a bug in matplotlib itself!
Cleaner input in the Sage notebook: