Skip to content

Fix data-trafo figure: replace empty [1,1.5) interval with two-interval partition#224

Merged
adibender merged 5 commits into
mainfrom
discrete-time-fix
Jun 16, 2026
Merged

Fix data-trafo figure: replace empty [1,1.5) interval with two-interval partition#224
adibender merged 5 commits into
mainfrom
discrete-time-fix

Conversation

@adibender

@adibender adibender commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • The PEM data transformation figure (fig-p4c22-data-trafo) used partition $\mathcal{A} = {0, 1, 1.5, 3}$, which produced an uninformative interval $[1, 1.5)$ with all-zero event indicators.
  • Replaced with $\mathcal{A} = {0, 1.5, 3}$ (two intervals), recalculating $t_{ij}$ and $t_j$ values throughout the figure.
  • Updated figure source (.drawio), re-exported PNG, and adapted surrounding text in 20-partition-reductions.qmd: partition reference, figure caption/alt-text, stacking comparison paragraph, and the "To make concrete…let…be" sentence.

…al partition {0,1.5,3}

Update the PEM data transformation figure and surrounding text to use
A={0,1.5,3} instead of A={0,1,1.5,3}, which had an uninformative empty
interval with all-zero event indicators. Recalculate t_ij and t_j values
accordingly. Adapt stacking comparison text and rewrite the awkward
"To make concrete... let ... be" sentence.
@adibender
adibender requested a review from RaphaelS1 June 15, 2026 15:40
Signed-off-by: Raphael Sonabend <raphaelsonabend@gmail.com>
@adibender
adibender merged commit 362784c into main Jun 16, 2026
1 check passed
@RaphaelS1
RaphaelS1 deleted the discrete-time-fix branch June 17, 2026 15:41
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