Ticket #3664 (closed enhancement: fixed)

Opened 3 months ago

Last modified 2 months ago

[with patch, positive review] major updates to root systems

Reported by: mhansen Assigned to: mhansen
Priority: major Milestone: sage-3.1
Component: combinatorics Keywords:
Cc:

Attachments

trac_3664-1.patch (205.7 kB) - added by mhansen on 08/06/2008 12:25:09 PM.
trac_3664-2.patch (145.2 kB) - added by mhansen on 08/06/2008 12:26:21 PM.
trac_3664-3.patch (5.3 kB) - added by mhansen on 08/08/2008 08:46:51 PM.

Change History

08/06/2008 12:25:09 PM changed by mhansen

  • attachment trac_3664-1.patch added.

08/06/2008 12:26:21 PM changed by mhansen

  • attachment trac_3664-2.patch added.

08/06/2008 12:26:55 PM changed by mhansen

  • summary changed from major updates to root systems to [with patch, positive review] major updates to root systems.

08/06/2008 12:30:05 PM changed by mhansen

This depends on #3662 and #3781 .

Credit goes to Dan Bump, Nicolas Thiery, Nicolas Borie (first contribution I believe), and Mike Hansen.

08/06/2008 06:27:57 PM changed by mhansen

  • milestone changed from sage-combinat to sage-3.1.

08/08/2008 04:07:50 PM changed by mabshoff

  • summary changed from [with patch, positive review] major updates to root systems to [with patch, positive review, needs work?] major updates to root systems.

I am seeing doctest failures here: 

sage -t -long devel/sage/sage/combinat/crystals/spins.py # 2 doctests failed
sage -t -long devel/sage/sage/combinat/crystals/letters.py # 3 doctests failed
sage -t -long devel/sage/sage/combinat/crystals/crystals.py # 13 doctests failed

For example: 

File "/scratch/mabshoff/release-cycle/sage-3.1.alpha1/tmp/spins.py", line 81:
    sage: len(TensorProductOfCrystals(C,C,generators=[[C.list()[0],C.list()[0]]]))
Exception raised:
    Traceback (most recent call last):
      File "/scratch/mabshoff/release-cycle/sage-3.1.alpha1/local/lib/python2.5/doctest.py", line 1228, in __run
        compileflags, 1) in test.globs
      File "<doctest __main__.example_1[4]>", line 1, in <module>
        len(TensorProductOfCrystals(C,C,generators=[[C.list()[Integer(0)],C.list()[Integer(0)]]]))###line 81:
    sage: len(TensorProductOfCrystals(C,C,generators=[[C.list()[0],C.list()[0]]]))
      File "/scratch/mabshoff/release-cycle/sage-3.1.alpha1/local/lib/python2.5/site-packages/sage/combinat/combinat.py", line 767, in __len__
        return self.count()
      File "/scratch/mabshoff/release-cycle/sage-3.1.alpha1/local/lib/python2.5/site-packages/sage/combinat/crystals/crystals.py", line 924, in count
        for x in self.highest_weight_vectors())
      File "/scratch/mabshoff/release-cycle/sage-3.1.alpha1/local/lib/python2.5/site-packages/sage/combinat/crystals/crystals.py", line 924, in <genexpr>
        for x in self.highest_weight_vectors())
      File "/scratch/mabshoff/release-cycle/sage-3.1.alpha1/local/lib/python2.5/site-packages/sage/combinat/crystals/tensor_product.py", line 365, in weight
        return sum(self[j].weight() for j in range(len(self))) 
      File "/scratch/mabshoff/release-cycle/sage-3.1.alpha1/local/lib/python2.5/site-packages/sage/combinat/crystals/tensor_product.py", line 365, in <genexpr>
        return sum(self[j].weight() for j in range(len(self)))
      File "/scratch/mabshoff/release-cycle/sage-3.1.alpha1/local/lib/python2.5/site-packages/sage/combinat/crystals/crystals.py", line 590, in weight
        return self.Phi() - self.Epsilon()
      File "/scratch/mabshoff/release-cycle/sage-3.1.alpha1/local/lib/python2.5/site-packages/sage/combinat/crystals/crystals.py", line 686, in Phi
        return sum(self.phi(i) * self._parent.Lambda()[i-1] for i in self.index_set())
      File "/scratch/mabshoff/release-cycle/sage-3.1.alpha1/local/lib/python2.5/site-packages/sage/combinat/crystals/crystals.py", line 686, in <genexpr>
        return sum(self.phi(i) * self._parent.Lambda()[i-1] for i in self.index_set())
      File "/scratch/mabshoff/release-cycle/sage-3.1.alpha1/local/lib/python2.5/site-packages/sage/combinat/family.py", line 387, in __getitem__
        return self.dictionary.__getitem__(i)
    KeyError: 0

Could it be that there is a patch missing in the dependency chain?

Cheers,

Michael

08/08/2008 08:46:51 PM changed by mhansen

  • attachment trac_3664-3.patch added.

08/08/2008 08:47:27 PM changed by mhansen

  • summary changed from [with patch, positive review, needs work?] major updates to root systems to [with patch, positive review] major updates to root systems.

I added trac_3664-3.patch that fixes the doctest failures.

08/09/2008 03:24:56 PM changed by mabshoff

  • status changed from new to closed.
  • resolution set to fixed.

Merged all three patches in Sage 3.1.alpha1