SupposeNot / RAMP

Research Assistant for Maps and Polytopes
4 stars 0 forks source link

3343Toroid behavior #116

Closed SupposeNot closed 2 years ago

SupposeNot commented 2 years ago

Documentation claims second input must be 0 or 1, but 0 is not accepted.

gap> 3343Toroid(3,1);
ReflexibleManiplex([ 3, 3, 4, 3 ], "(r0 r1 r2 r3 r2 r1 r4 r3 r2 r3 r4 r1 r2 r3 r2 r1)^3")
gap> 3343Toroid(3,0);
The value of k should be 1 or 2.
fail
SupposeNot commented 2 years ago

Same problem with 24CellToroid.

CunningGabe commented 2 years ago

Turns out the code was right and the documentation was wrong. (See ARP Section 6E). Fixed in 1f4fd9191ca15dfbed51791a632fdccf4d700940.