Ticket #12251 (closed enhancement: fixed)

Opened 17 months ago

Last modified 9 months ago

Implementation of Littelmann path model for crystals

Reported by: aschilling Owned by: sage-combinat
Priority: major Milestone: sage-5.3
Component: combinatorics Keywords: crystals, sd40
Cc: sage-combinat Work issues:
Report Upstream: N/A Reviewers: Anne Schilling, Mark Shimozono
Authors: Mark Shimozono, Anne Schilling Merged in: sage-5.3.beta1
Dependencies: Stopgaps:

Description (last modified by aschilling) (diff)

This patch implements the crystal graph using the Littelmann path model. The elements are LS paths generated from the straight-line path to a given weight. Also, the iterator function and lists in crystals is cleaned up.

Attachments

trac_12251-littelmann-as.patch Download (30.9 KB) - added by aschilling 10 months ago.

Change History

comment:1 Changed 12 months ago by aschilling

  • Status changed from new to needs_review
  • Reviewers set to Anne Schilling, Mark Shimozono
  • Description modified (diff)

comment:2 Changed 12 months ago by aschilling

Hi Mark,

The latest version of the patch is attached and ready for the final review (it is also on the sage-combinat queue).

Anne

comment:3 Changed 10 months ago by aschilling

  • Keywords crystals, sd40 added; crystals removed
  • Description modified (diff)
  • Milestone changed from sage-5.1 to sage-5.2

comment:4 Changed 10 months ago by mshimo

  • Status changed from needs_review to positive_review

comment:5 Changed 10 months ago by jdemeyer

  • Milestone changed from sage-5.2 to sage-5.3

comment:6 follow-up: ↓ 7 Changed 10 months ago by jdemeyer

  • Status changed from positive_review to needs_work

There is a doctest failure: 

sage -t --long "devel/sage/sage/categories/classical_crystals.py"
**********************************************************************
File "/usr/local/src/sage-5.2.rc1/devel/sage/sage/categories/classical_crystals.py", line 323:
    sage: TestSuite(fb4(1,0,1,0)).run(verbose = True)  # long time (8s on sage.math, 2011)
Expected:
    running ._test_an_element() . . . pass
    running ._test_category() . . . pass
    running ._test_elements() . . .
    Running the test suite of self.an_element()
      running ._test_category() . . . pass
      running ._test_eq() . . . pass
      running ._test_not_implemented_methods() . . . pass
      running ._test_pickling() . . . pass
      running ._test_stembridge_local_axioms() . . . pass
      pass
    running ._test_elements_eq() . . . pass
    running ._test_enumerated_set_contains() . . . pass
    running ._test_enumerated_set_iter_cardinality() . . . pass
    running ._test_enumerated_set_iter_list() . . .Enumerated set too big; skipping test; see ``self.max_test_enumerated_set_loop``
    pass
    running ._test_eq() . . . pass
    running ._test_fast_iter() . . . pass
    running ._test_not_implemented_methods() . . . pass
    running ._test_pickling() . . . pass
    running ._test_some_elements() . . . pass
    running ._test_stembridge_local_axioms() . . . pass
Got:
    running ._test_an_element() . . . pass
    running ._test_category() . . . pass
    running ._test_elements() . . .
      Running the test suite of self.an_element()
      running ._test_category() . . . pass
      running ._test_eq() . . . pass
      running ._test_not_implemented_methods() . . . pass
      running ._test_pickling() . . . pass
      running ._test_stembridge_local_axioms() . . . pass
      pass
    running ._test_elements_eq() . . . pass
    running ._test_enumerated_set_contains() . . . pass
    running ._test_enumerated_set_iter_cardinality() . . . pass
    running ._test_enumerated_set_iter_list() . . . pass
    running ._test_eq() . . . pass
    running ._test_fast_iter() . . . pass
    running ._test_not_implemented_methods() . . . pass
    running ._test_pickling() . . . pass
    running ._test_some_elements() . . . pass
    running ._test_stembridge_local_axioms() . . . pass
**********************************************************************

Changed 10 months ago by aschilling

comment:7 in reply to: ↑ 6 Changed 10 months ago by aschilling

Thank you Jeroen for catching this! It should be fixed in the new attached patch.

Anne

comment:8 Changed 10 months ago by aschilling

  • Status changed from needs_work to positive_review

comment:9 Changed 9 months ago by jdemeyer

  • Status changed from positive_review to closed
  • Resolution set to fixed
  • Merged in set to sage-5.3.beta1
Note: See TracTickets for help on using tickets.