-
Notifications
You must be signed in to change notification settings - Fork 182
problem with a new feature for matrix groups #6222
Copy link
Copy link
Open
Labels
kind: bugIssues describing general bugs, and PRs fixing themIssues describing general bugs, and PRs fixing themkind: bug: unexpected errorIssues describing bugs in which computation unexpectedly encounters an error, and PRs fixing themIssues describing bugs in which computation unexpectedly encounters an error, and PRs fixing themkind: bug: wrong resultIssues describing bugs that result in mathematically or otherwise wrong results, and PRs fixing themIssues describing bugs that result in mathematically or otherwise wrong results, and PRs fixing them
Metadata
Metadata
Assignees
Labels
kind: bugIssues describing general bugs, and PRs fixing themIssues describing general bugs, and PRs fixing themkind: bug: unexpected errorIssues describing bugs in which computation unexpectedly encounters an error, and PRs fixing themIssues describing bugs in which computation unexpectedly encounters an error, and PRs fixing themkind: bug: wrong resultIssues describing bugs that result in mathematically or otherwise wrong results, and PRs fixing themIssues describing bugs that result in mathematically or otherwise wrong results, and PRs fixing them
Since #6213 got merged, it is possible to construct some matrix groups in such a way that their generators are represented in a different way. For that, one can set the global option
ConstructingFilter. CurrentlyIsPlistMatrixRepis a supported non-default value.This feature is experimental, and it is clear that the new kinds of matrix groups will run into problems.
Here is one such problem:
The problem is that the function
NicomorphismFFMatGroupOnFullSpacecaches the maps it computes, see theIsIdenticalcheck above.The keys in this cache are the dimension and the size of the field.
Thus we run into problems as soon as these parameters do not suffice.
A straightforward solution is to add the representation of the matrices in the group and the field of definition (to be compared w.r.t. object identity) as parameters for the keys.
Do you have better ideas?