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

Shorter long tests for KR tableaux and rigged configruations. #15581

Closed
tscrim opened this issue Dec 24, 2013 · 4 comments
Closed

Shorter long tests for KR tableaux and rigged configruations. #15581

tscrim opened this issue Dec 24, 2013 · 4 comments

Comments

@tscrim
Copy link
Collaborator

tscrim commented Dec 24, 2013

As per this thread: https://groups.google.com/forum/?fromgroups=#!topic/sage-combinat-devel/WMk7H-J65qE

The technical reasoning:

In #13872, I implemented e0 and f0 operators for KR tableaux and rigged configurations by mapping to their corresponding KirillovReshetikhinCrystal element, then applying e0 and f0 (respecitvely) there, and then pulling back. Previously they just returned None. In _test_stembridge_local_axioms(), it checks to see that e(i) and e(j) are both not None, which now there is a longer check if that's true, and if so, then it must perform the Stembridge axioms check. This is why the tests now take significantly longer than before.

The proposed solution:

Make the crystals smaller.

Depends on #13872

CC: @sagetrac-sage-combinat @anneschilling @vbraun

Component: documentation

Keywords: long tests

Author: Travis Scrimshaw

Branch/Commit: public/combinat/rigged_configurations/speedups @ d585a44

Reviewer: Volker Braun

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

@tscrim

This comment has been minimized.

@tscrim
Copy link
Collaborator Author

tscrim commented Dec 24, 2013

comment:1

I made the really long tests now on (much) smaller crystals. I also moved one example to a better spot.

@vbraun
Copy link
Member

vbraun commented Dec 24, 2013

comment:2

lgtm, thanks!

@vbraun
Copy link
Member

vbraun commented Dec 24, 2013

Reviewer: Volker Braun

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