Skip to content

docs: correct skyline quadrature self-consistency claim - #868

Closed
ivan-aksamentov wants to merge 1 commit into
rustfrom
docs/coalescent-skyline-quadrature-contract
Closed

docs: correct skyline quadrature self-consistency claim#868
ivan-aksamentov wants to merge 1 commit into
rustfrom
docs/coalescent-skyline-quadrature-contract

Conversation

@ivan-aksamentov

Copy link
Copy Markdown
Member

docs/coalescent-skyline-quadrature-contract -> rust

Update:

The quadrature issue claimed the midpoint rule is exact for the piecewise-constant skyline, with no quadrature error. That holds only if the operative demographic function is taken to be the midpoint-snapped one. For the merger-time-midpoint boundaries the tool actually reports, the rule is self-consistent, not exact.

This corrects the wording to self-consistency and records the off-grid error characterization: the midpoint rule is exact on every lineage-count interval that contains no skyline boundary, while at most $n-1$ intervals straddle a boundary and each contributes an error bounded by its exposure times the inter-segment $T_c$ contrast [src].

Work items

  • Replace the "no quadrature error" claim with self-consistency and record the off-grid error bound [src]

Possible improvements

The quadrature issue claimed the midpoint rule is exact for the piecewise-constant skyline with no quadrature error. That holds only if the demographic function is the midpoint-snapped one. For the reported merger-time-midpoint boundaries the rule is self-consistent, not exact: exact on lineage-count intervals with no skyline boundary, with a bounded straddle error on the up-to n_points-1 intervals crossing a boundary. Record the off-grid error characterization.
@rneher

rneher commented Jul 23, 2026

Copy link
Copy Markdown
Member

We might want to replace this midpoint business. I'll look into it.

@ivan-aksamentov
ivan-aksamentov deleted the docs/coalescent-skyline-quadrature-contract branch July 24, 2026 15:25
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