Skip to content

Conversation

@hoheinzollern
Copy link
Member

@hoheinzollern hoheinzollern commented Oct 8, 2023

Motivation for this change

Work on Chernoff bound, a consequence of Markov's inequality

Things done/to do
  • added corresponding entries in CHANGELOG_UNRELEASED.md
  • added corresponding documentation in the headers
Compatibility with MathComp 2.0
  • I added the label TODO: HB port to make sure someone ports this PR to
    the hierarchy-builder branch or I already opened an issue or PR (please cross reference).
Automatic note to reviewers

Read this Checklist and put a milestone if possible.

@affeldt-aist affeldt-aist marked this pull request as ready for review November 1, 2023 14:11
Copy link
Member

@affeldt-aist affeldt-aist left a comment

Choose a reason for hiding this comment

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

This is good enough for me. Moreover, this is going to be used to prove a sampling lemma soon, so we'll have the opportunity to come back to it.

@affeldt-aist affeldt-aist added TODO: MC2 port This PR must be ported to mathcomp 2 now that the. Remove this label when the port is done. enhancement ✨ This issue/PR is about adding new features enhancing the library labels Nov 1, 2023
@affeldt-aist affeldt-aist merged commit c8adb03 into math-comp:master Nov 1, 2023
affeldt-aist added a commit to affeldt-aist/analysis that referenced this pull request Nov 10, 2023
* chernoff proof

---------

Co-authored-by: Reynald Affeldt <[email protected]>
proux01 pushed a commit that referenced this pull request Nov 10, 2023
* chernoff proof

---------

Co-authored-by: Reynald Affeldt <[email protected]>
@proux01 proux01 removed the TODO: MC2 port This PR must be ported to mathcomp 2 now that the. Remove this label when the port is done. label Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement ✨ This issue/PR is about adding new features enhancing the library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants