g.Order().sage() is only added to the list if it is finite

sagemath · Jan 1, 2024 · 28427c4 · 28427c4
1 parent b390e75
commit 28427c4
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/sage/groups/abelian_gps/abelian_group.py b/src/sage/groups/abelian_gps/abelian_group.py
@@ -1684,7 +1684,7 @@ def __init__(self, ambient, gens, names="f", category=None):
             Multiplicative Abelian subgroup isomorphic to C2 x C3 x Z
              generated by {a, b^2, c}
             sage: F.gens_orders()
-            (6, 0)
+            (3, 2, 0)
             sage: F.gens()
             (a, b^2, c)
             sage: F.order()
@@ -1710,7 +1710,8 @@ def __init__(self, ambient, gens, names="f", category=None):
         H = libgap(ambient).Subgroup(H_gens)
 
         invs = H.TorsionSubgroup().AbelianInvariants().sage()
-        gens_orders = [g.Order().sage() for g in H.TorsionSubgroup().GeneratorsOfGroup()]
+        gens_orders = [order_sage for g in H.GeneratorsOfGroup() if (order_sage := g.Order().sage()) is not infinity]
+
         rank = len([1 for g in H.GeneratorsOfGroup()
                     if g.Order().sage() is infinity])
         invs += [0] * rank