Skip to content

feat: Add automorphism group for antimagma#205

Open
limakzi wants to merge 2 commits intomainfrom
feature/204-add-authomorphism-group
Open

feat: Add automorphism group for antimagma#205
limakzi wants to merge 2 commits intomainfrom
feature/204-add-authomorphism-group

Conversation

@limakzi
Copy link
Collaborator

@limakzi limakzi commented Oct 13, 2025

This partially solves #204.

@limakzi limakzi self-assigned this Oct 13, 2025
Copilot AI review requested due to automatic review settings October 13, 2025 04:55
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds functionality to compute the automorphism group of a magma structure. This feature implements a method to find all automorphisms (structure-preserving bijective mappings) of a given magma.

  • Implements MagmaAutomorphismGroup function that computes automorphisms by checking all permutations
  • Adds comprehensive documentation with usage examples
  • Includes test coverage verifying automorphism properties

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
lib/properties.gd Declares the MagmaAutomorphismGroup attribute with documentation and examples
lib/properties.gi Implements the automorphism group computation algorithm
tst/test_properties_magma_automorphism_group.tst Adds tests verifying automorphism group properties

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@limakzi limakzi changed the title Feature/204 add authomorphism group feat: Add automorphism group for antimagma Oct 13, 2025
@limakzi limakzi force-pushed the feature/204-add-authomorphism-group branch from 0641e58 to 029f3e5 Compare October 13, 2025 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants