analysis icon indicating copy to clipboard operation
analysis copied to clipboard

Wochoice

Open Tragicus opened this issue 1 year ago • 3 comments

Motivation for this change

Adds a proof of Zorn's lemma, Hausdorff maximal principle and the well ordering principle. Contains a patch of contra that we may want to extract.

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

Reference: How to document

Reminder to reviewers

Tragicus avatar Feb 10 '24 18:02 Tragicus

Contains a patch of contra that we may want to extract.

Do you mean that it ought to be a different PR? (That may help the review process.)

affeldt-aist avatar Mar 05 '24 06:03 affeldt-aist

I would not go so far as "ought to", the fix is quite trivial, but if you would prefer a different PR, I can do it.

Tragicus avatar Mar 05 '24 07:03 Tragicus

the fix is quite trivial

Ok. Since there is a number of definitions that are added to contra.v and many canonicals, it didn't look trivial to me ^_^.

if you would prefer a different PR, I can do it.

I think that it is better (different concerns, two commits are maybe better also for the git log).

affeldt-aist avatar Mar 05 '24 08:03 affeldt-aist

@Tragicus could you rephrase your commit message so as to make

Co-Authored-By: Georges Gonthier <[email protected]>

a single line separated from the title by a white line, and verbatim (otherwise github does not pick it up)

CohenCyril avatar Mar 27 '24 15:03 CohenCyril

Argh, I accidentally merged this instead of https://github.com/math-comp/analysis/pull/1183 and I don't have push right on master to revert that, could someone do it, maybe @CohenCyril ?

proux01 avatar Mar 28 '24 09:03 proux01

Done with @affeldt-aist @Tragicus could you please reopen the PR, sorry again.

proux01 avatar Mar 28 '24 09:03 proux01

Can I also rebase and clean up now?

Tragicus avatar Mar 28 '24 09:03 Tragicus

Sure, thanks

proux01 avatar Mar 28 '24 09:03 proux01

Reopened at https://github.com/math-comp/analysis/pull/1198

Tragicus avatar Mar 28 '24 10:03 Tragicus