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

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-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 >

[sage-combinat-devel] Re: # 10744

2011-02-13 Thread Benjamin Jones
On Feb 12, 1:37 pm, 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: > I've looked over the plot code in WeightLatticeRealization. I think perhaps the two sets of plotting code co

[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)

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:

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

2011-02-11 Thread Daniel Bump
Nicolas 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 RootLatticeRealization. I see the plotting

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 RootLatt

[sage-combinat-devel] Re: # 10744

2011-02-11 Thread Benjamin Jones
On Feb 10, 1:34 pm, Daniel Bump 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() > > Frequently o