Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improvements for finitely presented groups and homomorphisms #5468

Merged
merged 7 commits into from
Jul 24, 2023

Conversation

hulpke
Copy link
Contributor

@hulpke hulpke commented Jun 29, 2023

Performance Improvements to the KB and abelian quotients of fp groups. Included is some size forcing for permutation homomorphisms

@hulpke hulpke added kind: enhancement Label for issues suggesting enhancements; and for pull requests implementing enhancements kind: bug: wrong result Issues describing bugs that result in mathematically or otherwise wrong results, and PRs fixing them release notes: to be added PRs introducing changes that should be (but have not yet been) mentioned in the release notes priority: high labels Jun 29, 2023
@hulpke hulpke force-pushed the fixes branch 3 times, most recently from a457e14 to 6d3e5ab Compare July 3, 2023 16:01
@hulpke hulpke added release notes: not needed PRs introducing changes that are wholly irrelevant to the release notes and removed release notes: to be added PRs introducing changes that should be (but have not yet been) mentioned in the release notes labels Jul 5, 2023
lib/kbsemi.gi Outdated Show resolved Hide resolved
Force size computation on certain permutation groups before constructing
homomorphisms. This what the group order canbe used to help with
constructing the homomorphism stabilizer chain.
Storing all pairs to be considered in the KB can end up very memory
intensive, once a few hundred rules are reached. Instead store these pairs
in more compact format. This also helps when rules get deleted.

Also added are minor hooks to test for the fellow-travelling property.
simpler code, and force normal form on quotient if it is infinite.

Delete rows of zeroes in relator matrix, but do not delete the whole matrix
@hulpke hulpke merged commit aafd490 into gap-system:master Jul 24, 2023
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: bug: wrong result Issues describing bugs that result in mathematically or otherwise wrong results, and PRs fixing them kind: enhancement Label for issues suggesting enhancements; and for pull requests implementing enhancements priority: high release notes: not needed PRs introducing changes that are wholly irrelevant to the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants