Opened 10 years ago

Last modified 5 years ago

#6249 new defect

graph1.plot() + graph2.plot() doesn't zorder correctly

Reported by: ncalexan Owned by: rlm
Priority: minor Milestone: sage-6.4
Component: graphics Keywords: graph plot z order overlay
Cc: ekirkman, rlm, kcrisman Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

Let G1 and G2 be arbitrary graphs. G1.plot() + G2.plot() and G2.plot() + G1.plot() (notice the ordering) look the same for me, ie I cannot make one graph appear above the other. What seems to happen is that the vertices are brought forward in the zorder, but it appears that this is done globally, not locally for the individual plot. I claim this is a bug.

Change History (9)

comment:1 Changed 10 years ago by mhampton

I'd just like to add that this bothers me too. I once tried to fix it and got confused and gave up.

comment:2 Changed 10 years ago by vbraun

I disagree. If you do not specify the z-order then there is no good way of automatically figuring out what to do. Amongst the multitude of possible behaviours you could even argue that G1.plot() + G2.plot() and G2.plot() + G1.plot() should be the same as addition is commutative.

Just do G1.plot(zorder=0) + G2.plot(zorder=1) or vice versa if you care about the z-ordering.

I recommend to close this bug.

comment:3 Changed 10 years ago by kcrisman

  • Cc kcrisman added
  • Report Upstream set to N/A

comment:4 Changed 9 years ago by kcrisman

  • Component changed from graph theory to graphics
  • Priority changed from major to minor

comment:5 Changed 8 years ago by kcrisman

#3251 seems to be related, though perhaps not a dup.

comment:6 Changed 6 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:7 Changed 6 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:8 Changed 5 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:9 Changed 5 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4
Note: See TracTickets for help on using tickets.