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

Enhancements: Solvable permutation groups and others #1046

Merged
merged 11 commits into from
Feb 16, 2017

Conversation

hulpke
Copy link
Contributor

@hulpke hulpke commented Jan 4, 2017

Next iteration of keeping my changes up-to-date with master

Tick all what applies to this pull request

  • Adds new features
  • Improves and extends functionality
  • Fixes Bugs

Write below the description of changes (for the release notes)

A bugfix in NaturalHomomorphismByIdeal for polynomial rings.
Improvements in handling solvable permutation groups.
The trivial group now is a member of the perfect groups library.
Improvements in using tabulated data for maximal subgroups.

@codecov-io
Copy link

codecov-io commented Jan 5, 2017

Codecov Report

Merging #1046 into master will decrease coverage by -0.01%.
The diff coverage is 61.68%.

@@            Coverage Diff             @@
##           master    #1046      +/-   ##
==========================================
- Coverage   57.39%   57.38%   -0.01%     
==========================================
  Files         434      434              
  Lines      230693   230972     +279     
==========================================
+ Hits       132403   132551     +148     
- Misses      98290    98421     +131
Impacted Files Coverage Δ
lib/onecohom.gi 62.95% <ø> (-0.22%)
lib/clashom.gi 31.97% <ø> (-0.19%)
lib/morpheus.gi 76.88% <ø> (-0.05%)
grp/perf.grp 87.98% <100%> (+0.29%)
lib/groebner.gi 56.61% <100%> (+0.2%)
grp/perf1.grp 94.9% <100%> (ø)
lib/permdeco.gi 35.37% <15.71%> (-4.1%)
lib/string.gi 23.62% <5.26%> (-0.6%)
lib/maxsub.gi 33.81% <63.63%> (+1.13%)
lib/dict.gi 66.12% <83.33%> (-0.22%)
... and 15 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8971a30...47de5e3. Read the comment docs.

Also a hook for permpcgs motivated by particular generators.
So far this always used a series of subgroups -- generic code. The new code
builds a permpcgs with Sims method and then uses sorted/unsorted pcgs code
to translate. This should give a global speedup.

Set group properties only if groups is generated by pcsequence.
@hulpke hulpke merged commit cb6d4d1 into gap-system:master Feb 16, 2017
@olexandr-konovalov
Copy link
Member

@hulpke please have a look at #1152 about errors that show up in teststandard

@olexandr-konovalov olexandr-konovalov added the release notes: added PRs introducing changes that have since been mentioned in the release notes label Jan 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release notes: added PRs introducing changes that have since been mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants