Skip to content

refactor: computations of primitive embeddings and extensions of ZZLat#5949

Draft
StevellM wants to merge 5 commits intooscar-system:masterfrom
StevellM:sm/gluing_factory
Draft

refactor: computations of primitive embeddings and extensions of ZZLat#5949
StevellM wants to merge 5 commits intooscar-system:masterfrom
StevellM:sm/gluing_factory

Conversation

@StevellM
Copy link
Copy Markdown
Member

Work in progress on improving the machinery for computing primitive embeddings/extensions of ZZLat. The main changes currently are:

  • Speed up of decision time (sometimes by a factor 40);
  • Avoid redundancy on large scale computations (one of the new applications that the current interface does not handle well);
  • Handle more input conditions (e.g. elementary divisors of a glue group);
  • Lots of micro-optimization and fine-tuning (still more to come).

I would say that currently, the draft contains approximately 45% of the final changes. Open a draft to keep track of current changes but also to give a working version to anyone interested.

Most of what is missing is user interface, corner cases, documentation and tests. All the part about equivariant extensions is also missing and will be added once the "plain" case is complete and stabilized. The overall new infrastructure is here and should stay as is in the future (modulo some fine-tuning).

Related to #5726

@StevellM StevellM added topic: number theory WIP NOT ready for merging optimization Simpler/more performant code or more/better tests release notes: to be added PRs introducing changes that should be (but have not yet been) mentioned in the release notes needs tests labels Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs tests optimization Simpler/more performant code or more/better tests release notes: to be added PRs introducing changes that should be (but have not yet been) mentioned in the release notes topic: number theory WIP NOT ready for merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant