Skip to content

setX -> setY; setM -> setX#1276

Merged
affeldt-aist merged 3 commits intomath-comp:masterfrom
affeldt-aist:setX_setY_setM
Aug 6, 2024
Merged

setX -> setY; setM -> setX#1276
affeldt-aist merged 3 commits intomath-comp:masterfrom
affeldt-aist:setX_setY_setM

Conversation

@affeldt-aist
Copy link
Member

@affeldt-aist affeldt-aist commented Aug 1, 2024

Motivation for this change

@CohenCyril @t6s @proux01 @Tragicus

Checklist
  • added corresponding entries in CHANGELOG_UNRELEASED.md
  • added corresponding documentation in the headers

Reference: How to document

Reminder to reviewers

@affeldt-aist affeldt-aist added this to the 1.3.0 milestone Aug 1, 2024
@affeldt-aist affeldt-aist requested review from CohenCyril and t6s August 1, 2024 03:54
+ definition `setX`, notation ``` `^` ```
+ lemmas `setX0`, `set0X`, `setXK`, `setXC`, `setXA`, `setIXl`, `mulrXr`,
`setX_def`, `setXE`, `setXU`, `setXI`, `setXD`, `setXCT`, `setCXT`, `setXTC`, `setTXC`
+ definition `setY`, notation ``` `^` ```
Copy link
Member

@CohenCyril CohenCyril Aug 1, 2024

Choose a reason for hiding this comment

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

I am also going to argue that ^ generally means exponential and I would rather associate notation B `^` A to set_fun A B. For setY maybe we can pick y for now?

Copy link
Member Author

Choose a reason for hiding this comment

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

Or `+` ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

FWIW, the notation for exclusive or in ssrbool is _ (+) _

@affeldt-aist
Copy link
Member Author

I went for `+` because it is shorter. @CohenCyril

@affeldt-aist affeldt-aist merged commit 6c0f39b into math-comp:master Aug 6, 2024
@affeldt-aist affeldt-aist deleted the setX_setY_setM branch August 6, 2024 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants