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

Task 52 #90

Merged
merged 19 commits into from
Jul 30, 2024
Merged

Task 52 #90

merged 19 commits into from
Jul 30, 2024

Conversation

nomeata
Copy link
Contributor

@nomeata nomeata commented Jul 24, 2024

Very first steps. Some things were easy generalization of code that was already there. Left some comments about things I'll have to think about.

Copy link
Owner

@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.

Also, please move the generic partial-order stuff to a separate file in ToMathlib.

Carleson/DiscreteCarleson.lean Outdated Show resolved Hide resolved
Carleson/DiscreteCarleson.lean Outdated Show resolved Hide resolved
lakefile.lean Outdated
Comment on lines 35 to 37
require loogle from git
"https://github.com/nomeata/loogle.git" @ "master"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Of course I’ll remove this from this branch when and if this PR is getting closer to being merged.

Copy link
Owner

@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.

I'm happy to merge this quickly, and you can open a PR on top of it to clean/finish things.

Carleson/ToMathlib/Height.lean Outdated Show resolved Hide resolved
next => simp
next z =>
simp only [Nat.cast_inj, Nat.cast_le]
-- no single tactic for goal `∀ x < n, ¬z = x) ↔ n ≤ z`?
Copy link
Owner

Choose a reason for hiding this comment

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

forall_lt_iff_le is close, but not quite there...

Carleson/ToMathlib/Height.lean Outdated Show resolved Hide resolved
@nomeata nomeata marked this pull request as ready for review July 29, 2024 22:42
@nomeata
Copy link
Contributor Author

nomeata commented Jul 29, 2024

Ok, for a quick-merge-cleanup-later this might be just about viable now.

@fpvandoorn fpvandoorn merged commit 65803e4 into fpvandoorn:master Jul 30, 2024
2 checks passed
@nomeata
Copy link
Contributor Author

nomeata commented Jul 30, 2024

Thanks for the cleanup, I was a bit in a rush, it seems. Also forgot to change with_coheight to withCoheight sorry for that.

@fpvandoorn
Copy link
Owner

No worries. The standards here are much lower than Mathlib, so when you have time, please make a follow-up PR fixing it :-)

@nomeata nomeata mentioned this pull request Jul 30, 2024
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.

2 participants