Skip to content

Add SpeedupDataPcHom#6449

Open
hulpke wants to merge 3 commits into
gap-system:masterfrom
hulpke:speedupc
Open

Add SpeedupDataPcHom#6449
hulpke wants to merge 3 commits into
gap-system:masterfrom
hulpke:speedupc

Conversation

@hulpke

@hulpke hulpke commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Implements a chunked caching scheme for evaluating bijective GroupGeneralMappingByPcgs homomorphisms on PC words. Splits the source pcgs into chunks, caching linear combinations of generator images per exponent pattern, with the bottom elementary abelian layer handled via a matrix over the corresponding prime field. Includes a corresponding ImagesRepresentative method that uses this cached data for a >2.5x speedup, per the included GAPDoc example.

@fingolfin fingolfin added topic: performance bugs or enhancements related to performance (improvements or regressions) topic: library labels Jul 1, 2026
@codecov

codecov Bot commented Jul 1, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 89.47368% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.99%. Comparing base (433541e) to head (c4cc950).

Files with missing lines Patch % Lines
lib/ghompcgs.gi 89.18% 12 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6449      +/-   ##
==========================================
- Coverage   79.01%   78.99%   -0.02%     
==========================================
  Files         685      646      -39     
  Lines      293519   279134   -14385     
  Branches     8667     7904     -763     
==========================================
- Hits       231924   220502   -11422     
+ Misses      59794    57112    -2682     
+ Partials     1801     1520     -281     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

hulpke added 3 commits July 2, 2026 15:33
Implements a chunked caching scheme for evaluating bijective
GroupGeneralMappingByPcgs homomorphisms on PC words. Splits the source
pcgs into chunks, caching linear combinations of generator images per
exponent pattern, with the bottom elementary abelian layer handled via
a matrix over the corresponding prime field. Includes a corresponding
ImagesRepresentative method that uses this cached data for a >2.5x
speedup, per the included GAPDoc example.
This excludes timing factor, as test overhead kills it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic: library topic: performance bugs or enhancements related to performance (improvements or regressions)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants