Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implementation of Littelmann path model for crystals #12251

Closed
anneschilling opened this issue Jan 2, 2012 · 8 comments
Closed

Implementation of Littelmann path model for crystals #12251

anneschilling opened this issue Jan 2, 2012 · 8 comments

Comments

@anneschilling
Copy link

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.

CC: @sagetrac-sage-combinat

Component: combinatorics

Keywords: crystals, sd40

Author: Mark Shimozono, Anne Schilling

Reviewer: Anne Schilling, Mark Shimozono

Merged: sage-5.3.beta1

Issue created by migration from https://trac.sagemath.org/ticket/12251

@anneschilling
Copy link
Author

Reviewer: Anne Schilling, Mark Shimozono

@anneschilling

This comment has been minimized.

@anneschilling
Copy link
Author

comment:2

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

@anneschilling

This comment has been minimized.

@anneschilling
Copy link
Author

Changed keywords from crystals to crystals, sd40

@anneschilling anneschilling modified the milestones: sage-5.1, sage-5.2 Jul 11, 2012
@jdemeyer jdemeyer modified the milestones: sage-5.2, sage-5.3 Jul 11, 2012
@jdemeyer
Copy link

comment:6

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

@anneschilling
Copy link
Author

comment:7

Attachment: trac_12251-littelmann-as.patch.gz

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

Anne

@jdemeyer
Copy link

Merged: sage-5.3.beta1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants