SupposeNot / RAMP

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

HoleLength bug #108

Closed SupposeNot closed 2 years ago

SupposeNot commented 2 years ago

gap> HoleLength(SnubCube()); Error, no method found! For debugging hints type ?Recovery from NoMethodFound Error, no 1st choice method found for `HoleLength' on 1 arguments at /Applications/Gap.app/Contents/gap/lib/methsel2.g:249 called from <function "HANDLE_METHOD_NOT_FOUND">( ) called from read-eval loop at stdin:55 type 'quit;' to quit to outer loop brk> quit;

Mixer2021 commented 2 years ago

Which hole length did you want?
This should return the lengths of j-holes: the lengths of orbits under r0 (r1 r2)^(j-1) r2.

CunningGabe commented 2 years ago

Should we make HoleLength(M) default to HoleLength(M,2)?

SupposeNot commented 2 years ago

Ahh, I misread the documentation, and thought that HoleLength(thing) just calculated the 2-holes... and I forgot anyone ever cares about other kinds. No change needed.