Skip to content

[Merged by Bors] - feat(Data/Set/Intervals/Image): Complete API#7146

Closed
YaelDillies wants to merge 8 commits intomasterfrom
maps_to_Icc
Closed

[Merged by Bors] - feat(Data/Set/Intervals/Image): Complete API#7146
YaelDillies wants to merge 8 commits intomasterfrom
maps_to_Icc

Conversation

@YaelDillies
Copy link
Copy Markdown
Contributor

@YaelDillies YaelDillies commented Sep 14, 2023

Dualise all existing lemmas and prove their strictly monotone versions.

The lemmas are grouped as

  • mapsTo, image_subset
    • On, not On
      • Monotone, Antitone, StrictMono, StrictAnti
        • Ixi, Iix, Ixx

Open in Gitpod

Dualise all existing lemmas and prove their strictly monotone versions.
@eric-wieser eric-wieser added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Sep 14, 2023
@YaelDillies YaelDillies added awaiting-review awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Sep 14, 2023
@eric-wieser eric-wieser added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Sep 14, 2023
@eric-wieser
Copy link
Copy Markdown
Member

I've put this back as awaiting author due to it not satisfying this section of the style guide; note this likely changed since you last read it, @YaelDillies!

@github-actions github-actions bot removed the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label Sep 14, 2023
@YaelDillies YaelDillies added awaiting-review and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Sep 14, 2023
Copy link
Copy Markdown
Member

@eric-wieser eric-wieser left a comment

Choose a reason for hiding this comment

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

Since you now wrote the majority of this file, can you add a sentence or two summarizing the lemmas in the module docstring?

@jcommelin jcommelin added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Sep 14, 2023
@YaelDillies YaelDillies added awaiting-review and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Sep 19, 2023
@sgouezel sgouezel added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Sep 26, 2023
Copy link
Copy Markdown
Member

@eric-wieser eric-wieser left a comment

Choose a reason for hiding this comment

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

bors d+

Please address the style issues above and below.

I'll take your word for it that this is the complete set.

@bors
Copy link
Copy Markdown

bors bot commented Sep 28, 2023

✌️ 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.

@ghost ghost added the delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). label Sep 28, 2023
@YaelDillies
Copy link
Copy Markdown
Contributor Author

bors merge

bors bot pushed a commit that referenced this pull request Sep 28, 2023
Dualise all existing lemmas and prove their strictly monotone versions.

The lemmas are grouped as
* `mapsTo`, `image_subset`
  * `On`, not `On`
    * `Monotone`, `Antitone`, `StrictMono`, `StrictAnti`
      * `Ixi`, `Iix`, `Ixx`




Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
@bors
Copy link
Copy Markdown

bors bot commented Sep 28, 2023

Pull request successfully merged into master.

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot changed the title feat(Data/Set/Intervals/Image): Complete API [Merged by Bors] - feat(Data/Set/Intervals/Image): Complete API Sep 28, 2023
@bors bors bot closed this Sep 28, 2023
@bors bors bot deleted the maps_to_Icc branch September 28, 2023 14:22
@YaelDillies YaelDillies removed the awaiting-author A reviewer has asked the author a question or requested changes. label Apr 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). t-order Order theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants