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

[Merged by Bors] - feat(group_theory/group_action/defs): vadd_assoc_class #15844

Closed
wants to merge 6 commits into from

Conversation

YaelDillies
Copy link
Collaborator

Additivize is_scalar_tower into a new class vadd_assoc_class.


I need this to additivize properly results about mul_action.stabilizer that are themselves needed for Kneser's theorem (whose statement is usually given using additive notation, so I really can't do without this).

vadd_assoc_class as a name doesn't really go with is_scalar_tower (what would? is_add_scalar_assoc_class?) but it does match smul_comm_class and vadd_comm_class. I already suggested renaming is_scalar_tower to smul_assoc_class on Zulip, and it would make even more sense after this PR.

Open in Gitpod

@YaelDillies YaelDillies added the awaiting-review The author would like community review of the PR label Aug 3, 2022
Copy link
Member

@fpvandoorn fpvandoorn left a comment

Choose a reason for hiding this comment

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

LGTM

please update to_additive.tr so that it automatically guesses the correct name (let me know if you need help with that).

@fpvandoorn
Copy link
Member

bors d+

@bors
Copy link

bors bot commented Aug 12, 2022

✌️ YaelDillies can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@github-actions github-actions bot added delegated The PR author may merge after reviewing final suggestions. and removed awaiting-review The author would like community review of the PR labels Aug 12, 2022
@YaelDillies
Copy link
Collaborator Author

I think I got that right? Given that the queue is empty right now, let's give it a try.

bors merge

bors bot pushed a commit that referenced this pull request Aug 12, 2022
Additivize `is_scalar_tower` into a new class `vadd_assoc_class`.
@bors
Copy link

bors bot commented Aug 12, 2022

Build failed:

@YaelDillies
Copy link
Collaborator Author

Yep, that works.

bors merge

bors bot pushed a commit that referenced this pull request Aug 12, 2022
Additivize `is_scalar_tower` into a new class `vadd_assoc_class`.
@bors
Copy link

bors bot commented Aug 12, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(group_theory/group_action/defs): vadd_assoc_class [Merged by Bors] - feat(group_theory/group_action/defs): vadd_assoc_class Aug 12, 2022
@bors bors bot closed this Aug 12, 2022
@bors bors bot deleted the vadd_assoc_class branch August 12, 2022 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated The PR author may merge after reviewing final suggestions.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants