Skip to content

Add GSet methods for blocks and related functions#4692

Merged
fingolfin merged 7 commits intooscar-system:masterfrom
mjrodgers:GSet_docs4
Mar 19, 2025
Merged

Add GSet methods for blocks and related functions#4692
fingolfin merged 7 commits intooscar-system:masterfrom
mjrodgers:GSet_docs4

Conversation

@mjrodgers
Copy link
Copy Markdown
Collaborator

@mjrodgers mjrodgers commented Mar 4, 2025

I have taken a stab at converting the PermGroup blocks method to take a GSet Omega as an argument, but having a little trouble figuring out the last bit.

Ideally we will return a new GSet of blocks that will have the same acting_group as Omega, and the blocks will be subsets of Omega. Currently we have GAP use the permutation action to return the blocks as sets of integers, and generate a GSet from that using on_sets. I'm hoping there is a natural way to compose on_sets with action_function(Omega), any thoughts on this? (Once this is working I will copy the approach for the other relevant functions).

@mjrodgers
Copy link
Copy Markdown
Collaborator Author

mjrodgers commented Mar 5, 2025

@fingolfin I added some rough implementations of the other methods, I will keep working to add documentation.

Is the return for maximal_blocks for PermGroup supposed to use on_sets? It seems like it should.

@fingolfin
Copy link
Copy Markdown
Member

I'm hoping there is a natural way to compose on_sets with action_function(Omega), any thoughts on this?

Hmm, good question, I am not sure. I mean of course if action_function(Omega) is on_points resp. ^ then then the "composition" is clear, but what if one starts with something custom? Hmm

In general of course the action can also be nested; e.g. the action on a block system may itself admit non-trivial blocks, which then would be sets-of-sets-of-something (with "something" the initial objects, e.g. integers).

I need to think about this, and perhaps we need to discuss it in person next week.

I added some rough implementations of the other methods, I will keep working to add documentation.

Thank you!

Is the return for maximal_blocks for PermGroup supposed to use on_sets? It seems like it should.

This sounds plausible.

@fingolfin
Copy link
Copy Markdown
Member

@mjrodgers this is still marked as draft, is there anything missing or is it "ready for review" (if so please press that button :-) )

@mjrodgers
Copy link
Copy Markdown
Collaborator Author

mjrodgers commented Mar 18, 2025 via email

@mjrodgers mjrodgers added the release notes: use title For PRs: the title of this PR is suitable for direct use in the release notes label Mar 19, 2025
@mjrodgers mjrodgers marked this pull request as ready for review March 19, 2025 13:19
@fingolfin fingolfin enabled auto-merge (squash) March 19, 2025 13:21
@fingolfin fingolfin added the enhancement New feature or request label Mar 19, 2025
@fingolfin fingolfin merged commit 7607b58 into oscar-system:master Mar 19, 2025
28 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request release notes: use title For PRs: the title of this PR is suitable for direct use in the release notes topic: groups

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants