-
-
Notifications
You must be signed in to change notification settings - Fork 660
Description
In RecursivelyEnumeratedSet_graded.breadth_first_search_iterator, use list for next_level instead of set in order to iterate over his elements in the same order on each run.
This fix python3 errors inthe combinat.root_system.weyl_group and root_lattice_realisations module.
But this modification causes some new doctests failures in weylgroup :
vklein@tuono:~/odk/sage$ sage -t --long src/sage/combinat/root_system/weyl_group.py
too many failed tests, not using stored timings
Running doctests with ID 2019-06-11-11-15-20-edaa5a63.
Git branch: HEAD
Using --optional=build,cmake,dochtml,memlimit,mpir,primecount,python2,sage
Doctesting 1 file.
sage -t --long src/sage/combinat/root_system/weyl_group.py
**********************************************************************
File "src/sage/combinat/root_system/weyl_group.py", line 1128, in sage.combinat.root_system.weyl_group.WeylGroup_permutation._coerce_map_from_
Failed example:
W.has_coerce_map_from(W5)
Expected:
False
Got:
True
...
File "src/sage/combinat/root_system/weyl_group.py", line 1157, in sage.combinat.root_system.weyl_group.WeylGroup_permutation.simple_reflection
Failed example:
**with W = WeylGroup(['A',4], implementation="permutation")**
W.simple_reflections()
Expected:
Finite family {1: (1,11)(2,6)(7,9)(8,10)(12,16)(17,19)(18,20),
2: (1,6)(2,12)(3,7)(5,8)(11,16)(13,17)(15,18),
3: (2,7)(3,13)(4,5)(6,9)(12,17)(14,15)(16,19),
4: (3,5)(4,14)(7,8)(9,10)(13,15)(17,18)(19,20)}
Got:
Finite family {1: (1,11)(2,5)(6,8)(9,10)(12,15)(16,18)(19,20),
2: (1,5)(2,12)(3,6)(7,9)(11,15)(13,16)(17,19),
3: (2,6)(3,13)(4,7)(5,8)(12,16)(14,17)(15,18),
4: (3,7)(4,14)(6,9)(8,10)(13,17)(16,19)(18,20)}
I need help to look if the two results of W.simple_reflections()
are both correct or not and what's the result of W.has_coerce_map_from(W5)
is supposed to be.
CC: @seblabbe @fchapoton @tscrim @nthiery
Component: python3
Stopgaps: #28351
Author: Vincent Klein, John Palmieri, Sébastien Labbé
Branch: f2c51a8
Reviewer: Vincent Klein, Markus Wageringel
Issue created by migration from https://trac.sagemath.org/ticket/27967