Bug report in function sage.groups.generic.bsgs

asked 2022-02-27 14:49:31 +0200

DuanYuFi gravatar image

updated 2022-02-27 15:36:34 +0200

slelievre gravatar image

In sage/groups/generic.py, line 468, multiple is missing some arguments.

That makes the bsgs calculation in customized group raise ValueError.

Adding the missing arguments is enough to fix this bug, like this:

c = op(inverse(b), multiple(a, lb,
                            operation=operation,
                            identity=identity,
                            inverse=inverse,
                            op=op))
edit retag flag offensive close merge delete

Comments

Welcome to Ask Sage! Thank you for your question.

slelievre gravatar imageslelievre ( 2022-02-27 15:31:56 +0200 )edit

Congratulations for spotting this bug. Can you also provide a simple example to illustrate it?

slelievre gravatar imageslelievre ( 2022-02-27 15:38:06 +0200 )edit
1

To submit a fix, you can send a merge request to the SageMath repository on GitLab:

or submit a branch to the SageMath repository on Sage Trac:

Would you like help to do that?

slelievre gravatar imageslelievre ( 2022-02-27 15:40:44 +0200 )edit

Oh yeah, thanks Mr./Mrs. Slelievre, I'll report it on GitLab

DuanYuFi gravatar imageDuanYuFi ( 2022-02-28 03:21:47 +0200 )edit
add a comment