Regression in cryst package #3670

Activity

1. olexandr-konovalov

   added

   topic: packagesissues or PRs related to package handling, or specific to a package (for packages w/o issue tracker)issues or PRs related to package handling, or specific to a package (for packages w/o issue tracker)

   on Sep 20, 2019
2. fingolfin commented on Sep 20, 2019

   

   fingolfin

   on Sep 20, 2019

   Member

   More actions

   It seems to hang in tst/cryst.tst line 270:

   gap> List( M{[2,5,7]}, x -> Orbit( G, x, OnPoints ) );;
   

   There, G is SpaceGroupOnRightIT(3,208,'1').

   I bisected the problem to commit 0c67c5a introduced in PR #3655 by me. I have not yet looked why this breaks. Of course we can just revert that commit, but it'd be good to understand what's going on.
3. gaehler commented on Sep 20, 2019

   

   gaehler

   on Sep 20, 2019

   Last edited by gaehler

   More actions

   Is due to a changed behaviour of Pcgs - see my comment in [gap-packages/cryst#14]
4. gaehler commented on Oct 25, 2019

   

   gaehler

   on Oct 25, 2019

   More actions

   Should be fixed in Cryst Release 4.1.21.
5. fingolfin

   closed this as completedon Nov 22, 2019