refactor: golf Gaussian/GaussianIntegral#38250
refactor: golf Gaussian/GaussianIntegral#38250yuanyi-350 wants to merge 2 commits intoleanprover-community:masterfrom
Gaussian/GaussianIntegral#38250Conversation
PR summary 700fcd702fImport 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/pr_summary/declarations_diff.sh <optional_commit>
## more verbose report:
./scripts/pr_summary/declarations_diff.sh long <optional_commit>The doc-module for No changes to technical debt.You can run this locally as
|
|
@yuanyi-350 Can I make two global requests to you --- for all such golfing PRs you have made (i.e., please also apply this to all your PRs waiting for review) and all future ones: Firstly, please provide a more precise PR description: say what the PR actually does (this is the case for this PR already, but not for other PRs) --- and make the general description in your first sentences much shorter: instead of you can say which has half the length, but the same information. (Arguably, even the first two sentences could and should be shortened: focus on the why of a change, not the what.) |
|
Secondly, please provide benchmarking numbers for your PR --- and gather these before making your PR. They should be available the moment your PR is ready for review. Thanks! Finally, let me share (for the sake of transparency) that we are discussing your series of PRs within the mathlib maintainers. Hopefully, we can provide a more detailed answer to your question "are these PRs desirable?" soon. (The above points are table stakes for the answer to be yes.) |
|
@grunweg Thank you for your suggestions, which are very helpful for improving our work.
Finally, may I ask for your Zulip ID? I would like to continue discussing some questions with you. |
You're very welcome to ask your questions publically --- this way others can help you also (and perhaps learn from answers if they wonder about the same thing)! Zulip is generally a friendly place; the mathlib4 stream is probably suitable. |
This PR selectively ports part of #37968 and only migrates the changes in:
Mathlib/Analysis/SpecialFunctions/Gaussian/GaussianIntegral.leanConcretely, this PR:
integral_gaussian_complex_Ioiby introducing the integrand as a localfand identifying theIic 0andIoi 0integrals viaintegral_comp_neg_Ioiintegral_add_compland the symmetry of the Gaussian integrand on the half-line