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] - chore: add some fun_prop attributes for continuity #10769

Closed
wants to merge 3 commits into from

Conversation

j-loreaux
Copy link
Collaborator


Open in Gitpod

@j-loreaux j-loreaux added awaiting-review The author would like community review of the PR easy < 20s of review time. See the lifecycle page for guidelines. t-topology Topological spaces, uniform spaces, metric spaces, filters labels Feb 20, 2024
@@ -978,6 +978,13 @@ theorem Continuous.comp_continuousOn {g : β → γ} {f : α → β} {s : Set α
hg.continuousOn.comp hf (mapsTo_univ _ _)
#align continuous.comp_continuous_on Continuous.comp_continuousOn

@[fun_prop]
theorem Continuous.comp_continuousOn'
Copy link
Collaborator

Choose a reason for hiding this comment

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

I just realized that fun_prop stores only one composition theorem for each function property. This is true for any lambda theorem (_id,_const,_comp,_pi,_apply). Thus this this theorem clashes with ContinuousOn.comp'' and ContinuousOn.comp'.

The current behavior is that only the latest lambda theorem is used. I think it is ok to keep it marked as fun_prop for now. I will add a support for multiple variants of lambda theorems, I already have this working for fun_trans so it should be quite easy.

@lecopivo
Copy link
Collaborator

The rest of the theorems should be fine.

Copy link
Collaborator

@YaelDillies YaelDillies left a comment

Choose a reason for hiding this comment

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

If Tomáš approves...

maintainer merge

Copy link

🚀 Pull request has been placed on the maintainer queue by YaelDillies.

Copy link
Contributor

@dupuisf dupuisf 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+

Mathlib/Topology/Algebra/Polynomial.lean Outdated Show resolved Hide resolved
@mathlib-bors
Copy link

mathlib-bors bot commented Feb 21, 2024

✌️ j-loreaux can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added delegated and removed awaiting-review The author would like community review of the PR labels Feb 21, 2024
Co-authored-by: Frédéric Dupuis <31101893+dupuisf@users.noreply.github.com>
Co-authored-by: Yaël Dillies <yael.dillies@gmail.com>
@j-loreaux
Copy link
Collaborator Author

bors merge

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Feb 21, 2024
@mathlib-bors
Copy link

mathlib-bors bot commented Feb 21, 2024

This PR was included in a batch that was canceled, it will be automatically retried

@mathlib-bors
Copy link

mathlib-bors bot commented Feb 21, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore: add some fun_prop attributes for continuity [Merged by Bors] - chore: add some fun_prop attributes for continuity Feb 21, 2024
@mathlib-bors mathlib-bors bot closed this Feb 21, 2024
@mathlib-bors mathlib-bors bot deleted the j-loreaux/fun_prop-markings-for-cfc branch February 21, 2024 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated easy < 20s of review time. See the lifecycle page for guidelines. maintainer-merge ready-to-merge This PR has been sent to bors. t-topology Topological spaces, uniform spaces, metric spaces, filters
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants