Build test on a PR is giving an error while code is wroking on my machine

50 views
Skip to first unread message

Ruchit Jagodara

unread,
Feb 28, 2024, 1:23:45 PMFeb 28
to sage-devel
Hey there!

I recently made a PR regarding the implementation of the minimum_generating_set function in polynomial time,#37481. However, in this PR, one of my build tests is failing and giving a error. I'm bit confused about how to solve this because I have never seen this kind of error before. Can somebody help me with this?

**********************************************************************
18File "src/sage/groups/group.pyx", line 275, in sage.groups.group.Group.minimum_generating_set
19Failed example:
20 sage: G = PermutationGroup([(1,2,3), (2,3), (4,5)])
21Exception raised:
22 Traceback (most recent call last):
23 File "/sage/src/sage/doctest/forker.py", line 712, in _run
24 self.compile_and_execute(example, compiler, test.globs)
25 File "/sage/src/sage/doctest/forker.py", line 1147, in compile_and_execute
26 exec(compiled, globs)
27 File "<doctest sage.groups.group.Group.minimum_generating_set[0]>", line 1, in <module>
28 sage: G = PermutationGroup([(Integer(1),Integer(2),Integer(3)), (Integer(2),Integer(3)), (Integer(4),Integer(5))])
29 ^
30 NameError: name 'G' is not defined
31**********************************************************************

Ricardo Buring

unread,
Feb 28, 2024, 1:31:45 PMFeb 28
to sage-devel
On Wednesday, February 28, 2024 at 7:23:45 PM UTC+1 Ruchit Jagodara wrote:
I recently made a PR regarding the implementation of the minimum_generating_set function in polynomial time,#37481. However, in this PR, one of my build tests is failing and giving a error. I'm bit confused about how to solve this because I have never seen this kind of error before. Can somebody help me with this?

To start, I would replace "sage: sage: G = ..." by just "sage: G = ..." in that example.

Ruchit Jagodara

unread,
Feb 28, 2024, 2:41:06 PMFeb 28
to sage-devel
Oh ! right, there should be only one 'sage:', thanks for your help !
Reply all
Reply to author
Forward
0 new messages