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(order): countable categoricity of dense linear orders #2860

Closed
wants to merge 18 commits into from

Conversation

dwarn
Copy link
Collaborator

@dwarn dwarn commented May 29, 2020

We construct an order isomorphism between any two countable, nonempty, dense linear orders without endpoints, using the back-and-forth method.


Blocked by #2850

@dwarn dwarn added the blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. label May 29, 2020
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/generic_cofilter.lean Outdated Show resolved Hide resolved
@bryangingechen bryangingechen added awaiting-author A reviewer has asked the author a question or requested changes and removed blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. labels Jun 9, 2020
@robertylewis
Copy link
Member

@dwarn is this still WIP?

@dwarn
Copy link
Collaborator Author

dwarn commented Oct 30, 2020

@robertylewis Sorry, I haven't done any Lean these past few months and sort of abandoned this PR.

I had a look just now to bring it up to date with current mathlib and also add a stub about order embeddings. This will prove for example that any countable ordinal embeds in the reals.

There's still a bit of work required to make this mergeable (move some lemmas as Bhavik suggested, finish construction of order embedding, naming, doc strings, ...).

@robertylewis
Copy link
Member

@dwarn Okay, great! In that case we'll leave the PR open and tagged as it is. Whenever you're ready, please change the label to "awaiting review."

@dwarn dwarn added awaiting-review The author would like community review of the PR and removed awaiting-author A reviewer has asked the author a question or requested changes labels Oct 31, 2020
@dwarn dwarn marked this pull request as ready for review October 31, 2020 13:50
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Show resolved Hide resolved
Copy link
Collaborator

@bryangingechen bryangingechen left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Collaborator

@bryangingechen bryangingechen left a comment

Choose a reason for hiding this comment

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

Sorry, a few more style nitpicks.

src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
@Vierkantor Vierkantor self-requested a review November 3, 2020 16:32
Co-authored-by: Bryan Gin-ge Chen <bryangingechen@gmail.com>
Copy link
Collaborator

@Vierkantor Vierkantor left a comment

Choose a reason for hiding this comment

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

Thank you for picking up this PR again! I have a few remarks, but overall it looks great.

src/order/countable_dense_linear_order.lean Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
src/order/countable_dense_linear_order.lean Outdated Show resolved Hide resolved
dwarn and others added 2 commits November 3, 2020 17:27
Co-authored-by: Anne Baanen <Vierkantor@users.noreply.github.com>
@dwarn dwarn requested a review from Vierkantor November 3, 2020 18:53
@Vierkantor
Copy link
Collaborator

Thank you 🎉

bors r+

@github-actions github-actions bot added ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) and removed awaiting-review The author would like community review of the PR labels Nov 3, 2020
@dwarn
Copy link
Collaborator Author

dwarn commented Nov 3, 2020

Thanks to all reviewers!

bors bot pushed a commit that referenced this pull request Nov 3, 2020
We construct an order isomorphism between any two countable, nonempty, dense linear orders without endpoints, using the back-and-forth method.
@bors
Copy link

bors bot commented Nov 4, 2020

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(order): countable categoricity of dense linear orders [Merged by Bors] - feat(order): countable categoricity of dense linear orders Nov 4, 2020
@bors bors bot closed this Nov 4, 2020
@bors bors bot deleted the countable-dense-linear-order branch November 4, 2020 00:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants