Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-25 Thread Anne Schilling
On 2/24/11 9:19 AM, Nicolas M. Thiery wrote: On Mon, Feb 14, 2011 at 12:00:32PM -0800, Anne Schilling wrote: I just realized a strange error after application of #10632. Viewing Kirillov-Reshetikhin crystals themselves works fine. However, viewing their tensor products is broken after the applic

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-25 Thread Nicolas M. Thiery
On Sat, Feb 12, 2011 at 07:08:35AM -0800, Daniel Bump wrote: > > - In the root and weight lattice, the default coordinate system > >should be given by the natural basis. I.e. in the root lattice in > >rank 2, alpha_1 should have coordinates (0,1) and alpha_2 (1,0). > > What basis is this?

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-25 Thread Nicolas M. Thiery
On Mon, Feb 14, 2011 at 12:00:32PM -0800, Anne Schilling wrote: > I just realized a strange error after application of #10632. > Viewing Kirillov-Reshetikhin crystals themselves works fine. > However, viewing their tensor products is broken after the application > of the patch. Without the patch, t

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-16 Thread Anne Schilling
I fixed this problem now by implementing a method __lt__ for KR crystals. The new extended patch (which includes some other functionalities as well) is posted on trac. Anne On 2/14/11 12:00 PM, Anne Schilling wrote: I just realized a strange error after application of #10632. Viewing Kirillov-R

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-14 Thread Anne Schilling
I just realized a strange error after application of #10632. Viewing Kirillov-Reshetikhin crystals themselves works fine. However, viewing their tensor products is broken after the application of the patch. Without the patch, this is fine. I am not quite sure, where this error comes from. Cheers,

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-13 Thread Anne Schilling
On 2/12/11 7:08 AM, Daniel Bump wrote: - In the root and weight lattice, the default coordinate system should be given by the natural basis. I.e. in the root lattice in rank 2, alpha_1 should have coordinates (0,1) and alpha_2 (1,0). What basis is this? (What would the basis be for

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-12 Thread Daniel Bump
> - In the root and weight lattice, the default coordinate system >should be given by the natural basis. I.e. in the root lattice in >rank 2, alpha_1 should have coordinates (0,1) and alpha_2 (1,0). What basis is this? (What would the basis be for C3?) One natural basis would be the fun

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-12 Thread Nicolas M. Thiery
On Sat, Feb 12, 2011 at 12:20:17AM -0800, Anne Schilling wrote: > On 2/11/11 1:18 PM, Nicolas M. Thiery wrote: > >On Fri, Feb 11, 2011 at 12:32:24PM -0800, Anne Schilling wrote: > >>Nicolas, is your patch ready for review? I saw on trac that there are > >>a couple of timeouts. > > > >Yup. Robert to

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-12 Thread Anne Schilling
On 2/11/11 1:18 PM, Nicolas M. Thiery wrote: On Fri, Feb 11, 2011 at 12:32:24PM -0800, Anne Schilling wrote: Nicolas, is your patch ready for review? I saw on trac that there are a couple of timeouts. Yup. Robert told me he would review it next week. Ok, so I shouldn't review the patch then?

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-11 Thread Nicolas M. Thiery
On Fri, Feb 11, 2011 at 12:32:24PM -0800, Anne Schilling wrote: > Nicolas, is your patch ready for review? I saw on trac that there are > a couple of timeouts. Yup. Robert told me he would review it next week. The timeouts are unrelated; I get the same ones without the patch. Cheers,

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-11 Thread Anne Schilling
Dear Dan, After the two patches (fresh downloads from trac, to be sure that I have the latest) #10723 and #10632, there are no problems in sage_object.pyx, which is where we would see pickle problems. There are also no failures in any of the crystal files. (I ran sage -t -long *.) I think that

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-11 Thread Daniel Bump
> Dear Dan and Nicolas! > > As I said, I posted a revised patch on trac which includes Nicolas' review > patch. > > I reinstalled the old crystal pickles and ran > > lolita-3:pickle_jar anne$ sage -t -force_lib > "devel/sage/sage/structure/sage_object.pyx" > sage -t -force_lib "devel/sage/sa

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-11 Thread Anne Schilling
Dear Dan and Nicolas! As I said, I posted a revised patch on trac which includes Nicolas' review patch. I reinstalled the old crystal pickles and ran lolita-3:pickle_jar anne$ sage -t -force_lib "devel/sage/sage/structure/sage_object.pyx" sage -t -force_lib "devel/sage/sage/structure/sage_ob

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-11 Thread Anne Schilling
On 2/10/11 9:27 AM, Nicolas M. Thiery wrote: On Wed, Feb 09, 2011 at 02:59:56PM -0800, Anne Schilling wrote: That being said, this could advocate for keeping CrystalOfTableaux as a class (using the classcall trick as we had discussed) in order to maintain backward compatibility on pickles (besid

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-10 Thread Nicolas M. Thiery
On Wed, Feb 09, 2011 at 02:59:56PM -0800, Anne Schilling wrote: > >That being said, this could advocate for keeping CrystalOfTableaux as > >a class (using the classcall trick as we had discussed) in order to > >maintain backward compatibility on pickles (beside the other > >advantages). Anne: I can

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-09 Thread Anne Schilling
I repeated everything as in my previous email. The pickling failure is in: _class__sage_combinat_crystals_tensor_product_CrystalOfTableaux_with_category_element_class__.sobj The pickles that are in tmp/pickle_jar-4.6.2.alpha4/ include the three pickles that are fixed by the procedure I described

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-09 Thread Nicolas M. Thiery
On Tue, Feb 08, 2011 at 10:45:03AM -0800, Daniel Bump wrote: > > Meanwhile, here is a relevant message from Mike Hansen: > > > > http://groups.google.com/group/sage-combinat-devel/msg/8b826e732d699cf0 > > I also found this useful web page: > > http://ask.sagemath.org/question/123/what-is-the-sta

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-09 Thread Nicolas M. Thiery
Hi Anne and Dan, On Thu, Feb 03, 2011 at 05:28:56AM -0800, Anne Schilling wrote: > Thank you for your comments! I posted a revised patch for #10632 on the > sage-combinat > queue and the trac server which incorporates the suggestions below as well as > Nicolas' suggestions regarding the l

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-09 Thread Nicolas M. Thiery
On Tue, Feb 08, 2011 at 11:06:49PM -0800, Anne Schilling wrote: > What is the procedure? Looking back at #8911 > http://trac.sagemath.org/sage_trac/ticket/8911 > the new pickle jar was just attached to the ticket. > Should we do the same thing this time? Yup. > >>I repeated everything as in my pr

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-09 Thread Anne Schilling
For #8911 the new pickle jar was just attached. So I added this file to #10632. Probably you can do the same for #7922 (but then the tickets might not commute if both change the pickles). Anne On 2/9/11 4:45 AM, bump wrote: I need to know this too, for #7922. Maybe Nicolas or Mike Hansen could

[sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-09 Thread bump
I need to know this too, for #7922. Maybe Nicolas or Mike Hansen could comment. On Feb 8, 11:06 pm, Anne Schilling wrote: > What is the procedure? Looking back at > #8911http://trac.sagemath.org/sage_trac/ticket/8911 > the new pickle jar was just attached to the ticket. > Should we do the same

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-08 Thread Anne Schilling
Dear Dan, Thanks so much for investigating all of this and your detailed description on how to update the pickle jar. What is the procedure? Looking back at #8911 http://trac.sagemath.org/sage_trac/ticket/8911 the new pickle jar was just attached to the ticket. Should we do the same thing this t

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-08 Thread Daniel Bump
I wrote: > This was wrong. (I clearly must have copied the wrong pickle.) > > I repeated everything as in my previous email. The pickling failure is in: > > _class__sage_combinat_crystals_tensor_product_CrystalOfTableaux_with_category_element_class__.sobj > > The pickles that are in tmp/pickle

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-08 Thread Daniel Bump
I wrote: > For me this fixed three out of four tests in sage/structure/sage_object.pyx. I > still get a failure with: > > _class__sage_combinat_crystals_tensor_product_CrystalOfTableauxGeneric_with_category_element_class__.sobj This was wrong. (I clearly must have copied the wrong pickle.) I r

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-08 Thread Daniel Bump
I wrote: > Meanwhile, here is a relevant message from Mike Hansen: > > http://groups.google.com/group/sage-combinat-devel/msg/8b826e732d699cf0 I also found this useful web page: http://ask.sagemath.org/question/123/what-is-the-standard-pickle-jar-for-and-how-do-i So assuming your shell is bas

[sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-08 Thread bump
> Let's make a deal then: I'll try to write soon a draft of patch > improving the documentation of unpickle_all to give detailed > explanations on the procedure. And I let you finalize this patch > according to your experience while you try out the procedure. OK! My main concern here is that #106

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-08 Thread Nicolas M. Thiery
On Mon, Feb 07, 2011 at 08:38:58PM -0800, Daniel Bump wrote: > I was hoping it was. Selecting precisely for which pickles we drop backward compatibility can only be our decision. And the release manager is already overloaded. > > See the >

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-07 Thread Daniel Bump
Nicolas wrote: > Yup, I guess we can safely update the pickle jar in that case > also. But that's not the release manager's job :-) I was hoping it was. > See the > documentation of sage.structure.sage_object.unpickle_all for how to > a

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-07 Thread Nicolas M. Thiery
On Mon, Feb 07, 2011 at 07:13:50AM -0800, Daniel Bump wrote: > There are some other failures after Anne's patch in > sage_object.pyx. > > I had a related query in this message from last September, > in connection with #7922. I think the answer is that the > release manager should rebuild the pick

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-07 Thread Anne Schilling
Do you have the latest version of #10632 since I changed this on Friday and made the KR crystal use the new CrystalOfTableaux code instead of a tensor product. Oops, I had overlooked that there is a new version of #10632. With the current versions of #10732, 10632 and 10485, the only failed test

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-07 Thread Daniel Bump
Anne wrote: > Do you have the latest version of #10632 since I changed this on Friday > and made the KR crystal use the new CrystalOfTableaux code instead of a > tensor product. Oops, I had overlooked that there is a new version of #10632. With the current versions of #10732, 10632 and 10485, th

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-07 Thread Anne Schilling
On 2/7/11 5:58 AM, Daniel Bump wrote: After Nicolas' revised #10723 and Anne's two patches there is one error after sage -t on doc/en/thematic_tutorials/lie/affine_crystals.rst. File "/home/bump/src/sage-4.6.2.alpha3/devel/sage-queue/doc/en/thematic_tutorials/lie/affine_crystals.rst", line 16

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-07 Thread Daniel Bump
There are some other failures after Anne's patch in sage_object.pyx. I had a related query in this message from last September, in connection with #7922. I think the answer is that the release manager should rebuild the pickle jar, but I am not certain of this. Perhaps Nicolas can comment? htt

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-07 Thread Daniel Bump
After Nicolas' revised #10723 and Anne's two patches there is one error after sage -t on doc/en/thematic_tutorials/lie/affine_crystals.rst. File "/home/bump/src/sage-4.6.2.alpha3/devel/sage-queue/doc/en/thematic_tutorials/lie/affine_crystals.rst", line 165: sage: K.classical_decomposition()

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-07 Thread Nicolas M. Thiery
On Sat, Feb 05, 2011 at 12:45:11PM -0800, Anne Schilling wrote: > >Besides, I don't remember looking at a crystal of with index > >set; but that might be just me. > > The edges are colored by i for i in I. Oops, I fumbled my typing. It should have read: ``I don't remember looking at the graph of

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-07 Thread Nicolas M. Thiery
On Sat, Feb 05, 2011 at 08:05:34PM +, Nicolas M. Thiery wrote: > On Thu, Feb 03, 2011 at 05:38:19PM -0800, Daniel Bump wrote: > > Then things sort of work to the extent that Anne's patch can be > > tried out. However color_by_label doesn't seem to work. For example, > > Nicolas suggested: > >

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-06 Thread Anne Schilling
On 2/6/11 1:05 PM, Daniel Bump wrote: I am not sure I understand this comment. For me this section starts as: "The KR crystals `B^{n,s}` for types `C_n^{(1)}` and `D^{(2)}_{n+1}` were excluded from the above discussion." Could you be more specific? OK, something interesting happens involving

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-06 Thread Daniel Bump
> I am not sure I understand this comment. For me this section starts as: > "The KR crystals `B^{n,s}` for types `C_n^{(1)}` and `D^{(2)}_{n+1}` were > excluded from the above discussion." > Could you be more specific? OK, something interesting happens involving jsmath and the cascading style sh

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-06 Thread Anne Schilling
Dear Dan, Thank you for looking at my patch! I am looking at #10485. I see Anne posted a new version overnight. Comment 1. I built it with the -j option to use the jsfonts. sage -docbuild thematic_tutorials -j html With the jsmath option the {1,2, ..., } in this line: B^{r,s} \cong B(

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-06 Thread Daniel Bump
I am looking at #10485. I see Anne posted a new version overnight. Comment 1. I built it with the -j option to use the jsfonts. sage -docbuild thematic_tutorials -j html With the jsmath option the {1,2, ..., } in this line: B^{r,s} \cong B(s\omega_r) \quad \text{as a \{1,2,\ldots,n\}-cry

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-06 Thread Anne Schilling
Hi Nicolas, Anne, Dan: do you mind if I focus on getting this patch to work on 4.6.2 without trying to make it simultaneously work with 4.6.1? This is of course fine. The only thing is that then it might be good to get the sage-combinat working with sage-4.6.2 (which it currently does not). C

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-05 Thread Anne Schilling
On 2/5/11 12:41 PM, Nicolas M. Thiery wrote: On Sat, Feb 05, 2011 at 12:30:19PM -0800, Anne Schilling wrote: Well, in general it is not clear how big the index set I is, so I did not want to preset a coloring. If I remember well, color_by_label picks up colors of his choice for the non specifi

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-05 Thread Nicolas M. Thiery
On Sat, Feb 05, 2011 at 12:30:19PM -0800, Anne Schilling wrote: > Well, in general it is not clear how big the index set I is, so I did not > want to preset a coloring. If I remember well, color_by_label picks up colors of his choice for the non specified ones (or at least it should), so that migh

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-05 Thread Anne Schilling
On 2/5/11 12:05 PM, Nicolas M. Thiery wrote: On Thu, Feb 03, 2011 at 05:38:19PM -0800, Daniel Bump wrote: Then things sort of work to the extent that Anne's patch can be tried out. However color_by_label doesn't seem to work. For example, Nicolas suggested: sage: K = KirillovReshetikhinCrystal(

[sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-05 Thread bump
> Anne, Dan: do you mind if I focus on getting this patch to work on > 4.6.2 without trying to make it simultaneously work with 4.6.1? This would seem to me to be the best way to do it. Dan -- You received this message because you are subscribed to the Google Groups "sage-combinat-devel" group

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-05 Thread Nicolas M. Thiery
On Thu, Feb 03, 2011 at 05:38:19PM -0800, Daniel Bump wrote: > Then things sort of work to the extent that Anne's patch can be > tried out. However color_by_label doesn't seem to work. For example, > Nicolas suggested: > > sage: K = KirillovReshetikhinCrystal(['A',3,1], 1,1) > sage: G = K.digraph(

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-05 Thread Nicolas M. Thiery
Hi Anne & Dan! On Fri, Feb 04, 2011 at 07:08:35AM -0800, Daniel Bump wrote: > Anne: > > Nicolas, could you have a look why #10723 does not apply to sage-4.6.2? > > For me with sage-4.6.1 everything applies and works. As Dan said, that's a trivial conflict with Rob's patch. An easy fix whi

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-04 Thread Daniel Bump
> Nicolas, could you have a look why #10723 does not apply to sage-4.6.2? > For me with sage-4.6.1 everything applies and works. I'm guessing that #9074 was merged and slightly clashes with #10723. It seems to have changes to the part of graph_latex.py where there is a conflict. However that doe

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-04 Thread Daniel Bump
> Nicolas, could you have a look why #10723 does not apply to sage-4.6.2? > For me with sage-4.6.1 everything applies and works. The patch itself is not at the trac page. What is at the trac page is a *link* to the patch and the description: "patch under construction on the Sage-combinat queue"

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-04 Thread Anne Schilling
A first version of the thematic tutorial for Kirillov-Reshetikhin crystals (and a slightly revised version for crystals) is ready for review. A copy of the html documentation built using sage -docbuild thematic_tutorials html is available here: http://www.math.ucdavis.edu/~anne/sage/lie/cryst

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-03 Thread Anne Schilling
Nicolas, could you have a look why #10723 does not apply to sage-4.6.2? For me with sage-4.6.1 everything applies and works. Thanks, Anne On 2/3/11 5:38 PM, Daniel Bump wrote: Anne wrote: The new patch now depends on #10723 which is available on the sage-combinat server. The patch for #

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-03 Thread Daniel Bump
Anne wrote: > The new patch now depends on #10723 which is available on the sage-combinat > server. The patch for #10723 does not apply cleanly to sage-4.6.2.alpha3. I took the patch from the combinat queue, but since it would be good to get #10632 merged in sage, I'm asking whether #10723 can

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-03 Thread Anne Schilling
Dear Dan and Nicolas, Thank you for your comments! I posted a revised patch for #10632 on the sage-combinat queue and the trac server which incorporates the suggestions below as well as Nicolas' suggestions regarding the latex options/printing for crystals. The new patch now depends on #10723 w

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-03 Thread Anne Schilling
On 2/3/11 12:40 AM, Nicolas M. Thiery wrote: On Thu, Feb 03, 2011 at 12:21:06AM -0800, Anne Schilling wrote: I should have insisted on the *further*. Can the user for example do: sage: K = KirillovReshetikhinCrystal(['A',3,1], 1,1) sage: G = K.digraph() sage: G.set_latex_optio

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-03 Thread Nicolas M. Thiery
On Thu, Feb 03, 2011 at 12:21:06AM -0800, Anne Schilling wrote: > >I should have insisted on the *further*. Can the user for example do: > > > > sage: K = KirillovReshetikhinCrystal(['A',3,1], 1,1) > > sage: G = K.digraph() > > sage: G.set_latex_options(color_by_label = {0:"black", 1

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-03 Thread Anne Schilling
On 2/2/11 11:58 PM, Nicolas M. Thiery wrote: On Wed, Feb 02, 2011 at 11:51:54PM -0800, Anne Schilling wrote: sage: K = KirillovReshetikhinCrystal(['A',3,1], 1,1) sage: G = K.digraph() sage: G.set_latex_options(some further option) sage: view(G, pdflatex=True, tightpage=Tr

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-02 Thread Nicolas M. Thiery
On Wed, Feb 02, 2011 at 11:51:54PM -0800, Anne Schilling wrote: > > sage: K = KirillovReshetikhinCrystal(['A',3,1], 1,1) > > sage: G = K.digraph() > > sage: G.set_latex_options(some further option) > > sage: view(G, pdflatex=True, tightpage=True) > > The second option actually

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-02 Thread Anne Schilling
Hi! Note: for consistency with other Sage objects, you may want to use the occasion to rename that method to _latex_, and set latex=_latex_ (it's currently the converse). Done! However, I have a question: How can I now pass along the **options to the view command in this setting? Be

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-02 Thread Daniel Bump
I'm looking at #10632. Comment 1. After the patch, I observe the following. Line 757 of tensor_product.py contains: shapes = tuple( tuple(shape) for shape in shapes ) I think here the making of a tuple is important because a tuple is immutable, a list is not. To illustrate, suppose take th

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-02 Thread Nicolas M. Thiery
Hi! On Wed, Feb 02, 2011 at 03:14:27PM -0800, Anne Schilling wrote: > I have now implemented the method > > def latex(self, **options): > r"""Returns the crystal graph as a latex string. This can be > exported > to a file with self.latex_file('filename').

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-02 Thread Anne Schilling
Hi Nicolas! Just reimplement Crystals.ParentMethods._latex_ to basically do the above: G=self.digraph() G.set_latex_options(format="dot2tex") return latex(G) At this occasion, it could provide some more options to set_latex_options, like the missing edge labels. And possibly

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-01 Thread Anne Schilling
On 2/1/11 8:42 AM, Nicolas M. Thiery wrote: On Tue, Feb 01, 2011 at 06:50:53AM -0800, Anne Schilling wrote: Thank you. Does your patch depend on other patches in the sage-combinat queue? I think the answer is yes, since moving it in the series file to before the crystal patches gives an error. W

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-01 Thread Nicolas M. Thiery
On Tue, Feb 01, 2011 at 06:50:53AM -0800, Anne Schilling wrote: > Thank you. Does your patch depend on other patches in the sage-combinat queue? > I think the answer is yes, since moving it in the series file to before > the crystal patches gives an error. Which ones? Would you include them > into

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-01 Thread Anne Schilling
On 2/1/11 5:00 AM, Nicolas M. Thiery wrote: On Tue, Feb 01, 2011 at 04:31:03AM -0800, Anne Schilling wrote: Could you open a trac ticket for it so I can refer to it? Done: http://trac.sagemath.org/sage_trac/ticket/10723 Thank you. Does your patch depend on other patches in the sage-combinat

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-01 Thread Nicolas M. Thiery
On Tue, Feb 01, 2011 at 04:31:03AM -0800, Anne Schilling wrote: > Could you open a trac ticket for it so I can refer to it? Done: http://trac.sagemath.org/sage_trac/ticket/10723 Cheers, Nicolas -- Nicolas M. Thiéry "Isil" http://Nicolas.Thiery.name/ -- You recei

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-01 Thread Anne Schilling
On 2/1/11 4:02 AM, Nicolas M. Thiery wrote: On Tue, Feb 01, 2011 at 02:21:14AM -0800, Anne Schilling wrote: How quickly could you get the graphs_color_by_label-nt.patch polished and merged? In principle, it should be just an hour or two of work. So it's all about prioritizing it with the rest

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-01 Thread Nicolas M. Thiery
On Tue, Feb 01, 2011 at 02:21:14AM -0800, Anne Schilling wrote: > How quickly could you get the graphs_color_by_label-nt.patch polished and > merged? In principle, it should be just an hour or two of work. So it's all about prioritizing it with the rest of the backlog. In other words: adding a dep

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-02-01 Thread Anne Schilling
On 1/29/11 2:13 PM, Nicolas M. Thiery wrote: On Sat, Jan 29, 2011 at 05:11:13AM -0800, Anne Schilling wrote: Following your suggestions I uploaded a new patch which addresses the issues of representation and latex methods for spin crystals. They now have a new method _repr_ using the signature r

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-01-31 Thread Anne Schilling
On 1/31/11 8:57 AM, Daniel Bump wrote: Anne wrote: If it is more convenient for you, I can include it into 10485. This might be more strategic for speed of getting it merged. I emailed you a revised .png file in a private message. Thank you. I included it in the working thematic tutorial p

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-01-31 Thread Daniel Bump
Anne wrote: > If it is more convenient for you, I can include it into 10485. This might be more strategic for speed of getting it merged. I emailed you a revised .png file in a private message. Dan -- You received this message because you are subscribed to the Google Groups "sage-combinat-de

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-01-30 Thread Anne Schilling
On 1/30/11 4:28 PM, bump wrote: Anne wrote: For trac 8442, how did you make the .png files that appear in you tutorial? Did you make them using sage? Most of them were made using metapost, I think. the 3 and the 4 in the tensor product need to be switched. Oops! I'll get you a corrected

[sage-combinat-devel] Re: tickets 10632 and 10485

2011-01-30 Thread bump
Anne wrote: > For trac 8442, how did you make the .png files that appear in you > tutorial? Did you make them using sage? Most of them were made using metapost, I think. > the 3 and the 4 in the tensor product need to be switched. Oops! I'll get you a corrected .png. I can either make another

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-01-29 Thread Nicolas M. Thiery
On Sat, Jan 29, 2011 at 05:11:13AM -0800, Anne Schilling wrote: > Following your suggestions I uploaded a new patch which addresses the > issues of representation and latex methods for spin crystals. > They now have a new method _repr_ using the signature representation. > I also implemented a _lat

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-01-29 Thread Anne Schilling
Dear Dan, I have a correction to ticket #8422: in the picture ../media/tensor.png the 3 and the 4 in the tensor product need to be switched. Anne -- You received this message because you are subscribed to the Google Groups "sage-combinat-devel" group. To post to this group, send email to sage-

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-01-29 Thread Anne Schilling
Dear Dan, For trac 8442, how did you make the .png files that appear in you tutorial? Did you make them using sage? Thank you, Anne -- You received this message because you are subscribed to the Google Groups "sage-combinat-devel" group. To post to this group, send email to sage-combinat-deve

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-01-29 Thread Anne Schilling
Hi Dan and Nicolas! Thank you for your comments! Following your suggestions I uploaded a new patch which addresses the issues of representation and latex methods for spin crystals. They now have a new method _repr_ using the signature representation. I also implemented a _latex_ method f

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-01-28 Thread Nicolas M. Thiery
Hi Dan! On Fri, Jan 28, 2011 at 10:10:48AM -0800, bump wrote: > Looking at the classical part of the #10632, this is a good advance > and will be very useful. However I'm wondering if the latex output > can be improved. > One could either (1) implement tableaux with half-boxes on the right

[sage-combinat-devel] Re: tickets 10632 and 10485

2011-01-28 Thread bump
Looking at the classical part of the #10632, this is a good advance and will be very useful. However I'm wondering if the latex output can be improved. One could either (1) implement tableaux with half-boxes on the right edge or (2) figure out some way that the labels don't overlap. For example, t

Re: [sage-combinat-devel] Re: tickets 10632 and 10485

2011-01-27 Thread Anne Schilling
On 1/27/11 2:39 PM, bump wrote: I'll look at the patch. Thank you. I just posted a new patch on trac for ticket 10632 which needs reviewhttp://trac.sagemath.org/sage_trac/ticket/10632 Besides the implementation of some missing Kirillov-Reshetikhin crystals, it also implements spin crystal ta

[sage-combinat-devel] Re: tickets 10632 and 10485

2011-01-27 Thread bump
I'll look at the patch. > I just posted a new patch on trac for ticket 10632 which needs > reviewhttp://trac.sagemath.org/sage_trac/ticket/10632 > Besides the implementation of some missing Kirillov-Reshetikhin crystals, > it also implements spin crystal tableaux. > Any volunteers? > > Dan had al