Skip to content

add poisson distribution#1668

Merged
affeldt-aist merged 2 commits into
math-comp:masterfrom
IshiguroYoshihiro:poisson_20250702
Jul 3, 2025
Merged

add poisson distribution#1668
affeldt-aist merged 2 commits into
math-comp:masterfrom
IshiguroYoshihiro:poisson_20250702

Conversation

@IshiguroYoshihiro

Copy link
Copy Markdown
Collaborator
Motivation for this change

Adding Poisson distribution.

Checklist
  • added corresponding entries in CHANGELOG_UNRELEASED.md
  • added corresponding documentation in the headers

Reference: How to document

Merge policy

As a rule of thumb:

  • PRs with several commits that make sense individually and that
    all compile are preferentially merged into master.
  • PRs with disorganized commits are very likely to be squash-rebased.
Reminder to reviewers

@affeldt-aist affeldt-aist self-requested a review July 3, 2025 03:44

@affeldt-aist affeldt-aist left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Very welcomed addition!

esum_limn_ge0 can actually be proved as follows:

Local Open Scope classical_set_scope.
Local Open Scope ring_scope.
Variables (R : realType).
Context {f : nat -> R}.
Hypothesis (f0 : forall n, 0 <= f n).

Lemma esum_limn_ge0 :
 \esum_(i in [set: nat]) (f i)%:E
  = limn
      (EFin \o (fun n : nat => (\sum_(0 <= k < n) f k)%R)).
Proof.
rewrite -set_true -nneseries_esum//; last by move=> *; rewrite lee_fin.
by apply/congr_lim/funext => n/=; rewrite -sumEFin.
Qed.

End elim_limn_ge0.

which suggests that the lemma is moderately useful.
In fact, it is a bit shorter to prove poisson_setT
by moving from esum to nneseries where there are
more lemmas (and this proof does not require esum_limn_ge0).
I have therefore removed it.
On the other hand, this also suggests that nneseries_esum
is a bit awkward to use when use with the whole set, I have
therefore added the technical lemma nneseries_esumT
and this helps a bit imo.

@affeldt-aist affeldt-aist merged commit 19aae45 into math-comp:master Jul 3, 2025
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants