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

feat(analysis/mean_inequalities) : Prove AM-GM #1836

Merged
merged 8 commits into from
Feb 19, 2020
Merged

feat(analysis/mean_inequalities) : Prove AM-GM #1836

merged 8 commits into from
Feb 19, 2020

Conversation

urkud
Copy link
Member

@urkud urkud commented Dec 28, 2019

Should be ready now. I can't guess what versions will be useful in practice, so I prove a few versions, and leave the rest to someone who'll want to actually use these inequalities.

@urkud urkud added blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. WIP Work in progress and removed blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. labels Dec 28, 2019
@urkud urkud changed the title WIP feat(analysis/mean_inequalities) : Prove AM-GM feat(analysis/mean_inequalities) : Prove AM-GM Jan 3, 2020
@urkud urkud added this to In progress in Calculus / analysis Jan 3, 2020
@urkud urkud added awaiting-review The author would like community review of the PR and removed WIP Work in progress labels Feb 19, 2020
src/analysis/convex/specific_functions.lean Outdated Show resolved Hide resolved
src/analysis/mean_inequalities.lean Outdated Show resolved Hide resolved
src/analysis/mean_inequalities.lean Outdated Show resolved Hide resolved
src/analysis/mean_inequalities.lean Outdated Show resolved Hide resolved
open real finset
open_locale classical nnreal

variables {ι : Type u} (s : finset ι)
Copy link
Collaborator

Choose a reason for hiding this comment

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

s could be implicit in most statements (all?)

Copy link
Member Author

Choose a reason for hiding this comment

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

When I specialize the theorems for 2 and 3 variables I explicitly supply univ because I have w₁+w₂=1, not univ.sum w = 1.

src/analysis/mean_inequalities.lean Outdated Show resolved Hide resolved
src/analysis/mean_inequalities.lean Outdated Show resolved Hide resolved
@sgouezel sgouezel added awaiting-author A reviewer has asked the author a question or requested changes and removed awaiting-review The author would like community review of the PR labels Feb 19, 2020
urkud and others added 4 commits February 19, 2020 13:09
Co-Authored-By: sgouezel <sebastien.gouezel@univ-rennes1.fr>
Co-Authored-By: sgouezel <sebastien.gouezel@univ-rennes1.fr>
@urkud urkud added awaiting-review The author would like community review of the PR and removed awaiting-author A reviewer has asked the author a question or requested changes labels Feb 19, 2020
Co-Authored-By: Johan Commelin <johan@commelin.net>
Calculus / analysis automation moved this from In progress to Reviewer approved Feb 19, 2020
@sgouezel sgouezel added ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) and removed awaiting-review The author would like community review of the PR labels Feb 19, 2020
@mergify mergify bot merged commit 2d6556d into master Feb 19, 2020
Calculus / analysis automation moved this from Reviewer approved to Done Feb 19, 2020
@mergify mergify bot deleted the AM-GM branch February 19, 2020 23:21
anrddh pushed a commit to anrddh/mathlib that referenced this pull request May 15, 2020
…1836)

* feat(analysis/mean_inequalities) : Prove AM-GM

* Update, add more inequalities

* Update src/analysis/convex/specific_functions.lean

Co-Authored-By: sgouezel <sebastien.gouezel@univ-rennes1.fr>

* Update src/analysis/mean_inequalities.lean

Co-Authored-By: sgouezel <sebastien.gouezel@univ-rennes1.fr>

* Update src/analysis/mean_inequalities.lean

* Small fixes, thanks @sgouezel

* Update src/analysis/mean_inequalities.lean

Co-Authored-By: Johan Commelin <johan@commelin.net>

Co-authored-by: sgouezel <sebastien.gouezel@univ-rennes1.fr>
Co-authored-by: Johan Commelin <johan@commelin.net>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
anrddh pushed a commit to anrddh/mathlib that referenced this pull request May 16, 2020
…1836)

* feat(analysis/mean_inequalities) : Prove AM-GM

* Update, add more inequalities

* Update src/analysis/convex/specific_functions.lean

Co-Authored-By: sgouezel <sebastien.gouezel@univ-rennes1.fr>

* Update src/analysis/mean_inequalities.lean

Co-Authored-By: sgouezel <sebastien.gouezel@univ-rennes1.fr>

* Update src/analysis/mean_inequalities.lean

* Small fixes, thanks @sgouezel

* Update src/analysis/mean_inequalities.lean

Co-Authored-By: Johan Commelin <johan@commelin.net>

Co-authored-by: sgouezel <sebastien.gouezel@univ-rennes1.fr>
Co-authored-by: Johan Commelin <johan@commelin.net>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants