Re: [sage-combinat-devel] Re: # 10744

2011-02-25 Thread Nicolas M. Thiery
On Sat, Feb 12, 2011 at 11:37:00AM -0800, Benjamin Jones wrote: It sounds to me like I should remove my `plot` method from Trac #10744, leaving the demazure character there, and start a new ticket to: a) move the existing plot code there to RootLatticeRealization b) update that code in

Re: [sage-combinat-devel] Re: # 10744

2011-02-25 Thread Nicolas M. Thiery
On Sun, Feb 13, 2011 at 10:17:54AM -0800, Benjamin Jones wrote: I've looked over the plot code in WeightLatticeRealization. I think perhaps the two sets of plotting code could co-exist, one in RootLatticeRealization (or wherever) and mine in WeylCharacterRing. The two sets of code have some

[sage-combinat-devel] Re: # 10744

2011-02-13 Thread Benjamin Jones
On Feb 12, 1:37 pm, Benjamin Jones benjaminfjo...@gmail.com wrote: It sounds to me like I should remove my `plot` method from Trac #10744, leaving the demazure character there, and start a new ticket to: I've looked over the plot code in WeightLatticeRealization. I think perhaps the two sets

Re: [sage-combinat-devel] Re: # 10744

2011-02-13 Thread Daniel Bump
Benjamine Jones wrote: I've looked over the plot code in WeightLatticeRealization. I think perhaps the two sets of plotting code could co-exist, one in RootLatticeRealization (or wherever) and mine in WeylCharacterRing. The two sets of code have some things in common, but the plots are

Re: [sage-combinat-devel] Re: # 10744

2011-02-12 Thread Nicolas M. Thiery
On Fri, Feb 11, 2011 at 02:26:03PM -0800, Daniel Bump wrote: I see the plotting code is in WeightLatticeRealization, added over a year ago in #4326. Apparently it was written by Nicolas Borie. Yup, Nicolas did the partial porting. It should be moved to RootLatticeRealization. A couple notes:

[sage-combinat-devel] Re: # 10744

2011-02-12 Thread Benjamin Jones
Yup, Nicolas did the partial porting. It should be moved to RootLatticeRealization. It sounds to me like I should remove my `plot` method from Trac #10744, leaving the demazure character there, and start a new ticket to: a) move the existing plot code there to RootLatticeRealization b)

[sage-combinat-devel] Re: # 10744

2011-02-11 Thread Benjamin Jones
On Feb 10, 1:34 pm, Daniel Bump b...@match.stanford.edu wrote: Here are some comments on #10744. Very nice! I do have some comments. Thanks, and I appreciate the feedback! sage: A2=WeylCharacterRing(A2,style=coroots) sage: a2=WeightRing(A2) sage: rp=a2(A2(2,1)) sage: rp.plot()

Re: [sage-combinat-devel] Re: # 10744

2011-02-11 Thread Nicolas M. Thiery
On Fri, Feb 11, 2011 at 12:28:12PM -0800, Benjamin Jones wrote: Where do people think the main code for plotting should go? In WeightRingElement, WeylCharacterRing, or in weyl_character.py but outside of any class (and have plot methods in both classes that call this function?) In