feat(Manifold/MFDeriv): add fun_prop to MDifferentiable#33808
feat(Manifold/MFDeriv): add fun_prop to MDifferentiable#33808seewoo5 wants to merge 1 commit intoleanprover-community:masterfrom
MDifferentiable#33808Conversation
PR summary cb3ccde1edImport changes for modified filesNo significant changes to the import graph Import changes for all files
Declarations diffNo declarations were harmed in the making of this PR! 🐙 You can run this locally as follows## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>
## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>The doc-module for No changes to technical debt.You can run this locally as
|
|
Hi! Thank you for making this PR. Indeed, In the mean-time, I am not sure if we want to merge this PR. Tagging the comp lemma (or in general, tagging this property with fun_prop) may induce the expectation that fun_prop just works here, which is decidedly not the case. I'd rather fix this properly, than land a half-baked solution now and deal with the user confusion. Can you live with maintaining these tags (ideally, with some warning that this is not robust support yet) in the sphere-packing repository for a little while? Once fun_prop support has improved, I'll gladly review a PR making these tags. |
|
Regarding this PR itself: if you're tagging MDifferentiable, I would also expect tags on MDifferentiableAt, and presumably also MDifferentiableOn and MDifferentiableWithinAt (to match what we do for Differentiable). That's a secondary concern compared to my previous comment. |
Thank you, I wasn't aware of that. Does it mean that I can still fun_prop-ing for specific model with corners? For the actual application on sphere packing side, every function will be a modularform-like objects with same domain (upper half plane) and codomain (C). I believe I can do this on sphere packing repo (probably put fun_prop there?), and then remove them when everything is ready and this PR got merged.
Yes, I agree with it. I can also take account this later. In that case, is it still makes sense to add |
If |
|
Since we have |
|
(At some point, we should have automation to verify that these sets of lemmas are in sync. But that's a different matter.) |
I made #33830 ! |
|
This pull request has conflicts, please merge |
Add FunProp to
MDifferentiable. For example, the following works (whenF'is normed𝕜-algebra)Motivated from sphere packing project.