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
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?
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
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
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,
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
> - 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
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
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?
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,
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
> 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
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
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
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
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
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
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
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
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
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
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
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
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
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
> 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
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
>
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
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
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
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
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
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
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()
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
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:
> >
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
> 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
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(
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
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
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
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
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(
> 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
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(
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
> 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
> 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"
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
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 #
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
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
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
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
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
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
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
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
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').
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
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
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
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
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
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
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
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
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
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
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
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
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
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-
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
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
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
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
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
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
80 matches
Mail list logo