Skip to content

chore(*): s/continuity/fun_prop/#12661

Closed
urkud wants to merge 18 commits intomasterfrom
YK-continuity
Closed

chore(*): s/continuity/fun_prop/#12661
urkud wants to merge 18 commits intomasterfrom
YK-continuity

Conversation

@urkud
Copy link
Copy Markdown
Member

@urkud urkud commented May 4, 2024


Open in Gitpod

@ghost ghost added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label May 16, 2024
@ghost ghost removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label May 21, 2024
@ghost ghost added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label May 24, 2024
mathlib-bors bot pushed a commit that referenced this pull request Jun 19, 2024
Inspired by #12661: let's start with some easy instances, so we can focus on the difficult ones.

In particular, this PR does not depend on any in-progress bugfixes, not tries to change a default value `by continuity` to `by fun_prop`.

I am not at all confident about the tagging of lemmas with fun_prop: I remember this being subtle, but can't find any documentation on this. @lecopivo Can you advise on these lemmas (or simply link to the right documentation if I just overlooked it)?
AntoineChambert-Loir pushed a commit that referenced this pull request Jun 20, 2024
Inspired by #12661: let's start with some easy instances, so we can focus on the difficult ones.

In particular, this PR does not depend on any in-progress bugfixes, not tries to change a default value `by continuity` to `by fun_prop`.

I am not at all confident about the tagging of lemmas with fun_prop: I remember this being subtle, but can't find any documentation on this. @lecopivo Can you advise on these lemmas (or simply link to the right documentation if I just overlooked it)?
grunweg added a commit that referenced this pull request Jun 20, 2024
Inspired by #12661: let's start with some easy instances, so we can focus on the difficult ones.

In particular, this PR does not depend on any in-progress bugfixes, not tries to change a default value `by continuity` to `by fun_prop`.

I am not at all confident about the tagging of lemmas with fun_prop: I remember this being subtle, but can't find any documentation on this. @lecopivo Can you advise on these lemmas (or simply link to the right documentation if I just overlooked it)?
@grunweg
Copy link
Copy Markdown
Contributor

grunweg commented Jun 23, 2024

Just in case you didn't see this, I also worked in this direction in the interim: #14009, #13959 and #13994 have some "easy" fixes (just replacements, and some surrounding golfs) and can land already.
Sorry for duplicating work; I would be happy to see either PR land.

@grunweg
Copy link
Copy Markdown
Contributor

grunweg commented Jun 23, 2024

#13869 is a small chore necessary for using fun_prop in measure theory; the fun_prop basically approved this, but this needs a maintainer's look.

kbuzzard pushed a commit that referenced this pull request Jun 26, 2024
Inspired by #12661: let's start with some easy instances, so we can focus on the difficult ones.

In particular, this PR does not depend on any in-progress bugfixes, not tries to change a default value `by continuity` to `by fun_prop`.

I am not at all confident about the tagging of lemmas with fun_prop: I remember this being subtle, but can't find any documentation on this. @lecopivo Can you advise on these lemmas (or simply link to the right documentation if I just overlooked it)?
@urkud
Copy link
Copy Markdown
Member Author

urkud commented Aug 12, 2024

Closing, we have better progress in other PRs now.

@urkud urkud closed this Aug 12, 2024
@urkud urkud deleted the YK-continuity branch August 12, 2024 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants