perf(export): rasteriseer PDF/PPTX met een begrensd geheugenvenster #1047

New issue

Closed

opened 2026-08-01 08:26:06 +00:00 by brenno · 1 comment

brenno commented

2026-08-01 08:26:06 +00:00

Owner

Context
SlideRasterizer bewaart alle gerenderde PNGs in List en verhoogt tegelijk de globale image-cache tot 1 GiB. PDF/PPTX-opbouw start pas nadat alle slides zijn gerasteriseerd.

Impact
Bij 1920×1080 is één ruwe capture circa 7,9 MiB. Honderden hoog-entropische slides kunnen meerdere gigabytes aan renders vasthouden, nog vóór PDF/ZIP-assemblage.

Relevante code

lib/services/slide_rasterizer.dart:92 en 123
lib/services/export_service.dart:477 en 527

Acceptatie

Rasteriseer en schrijf met een klein expliciet begrensd venster, of weiger vooraf op een onderbouwd totaalbudget.
Annuleren ruimt tussenresultaten en image-cache altijd op.
Een schaaltest met een groot deck bewaakt piekgeheugen/gelijktijdig levende renders, niet alleen dat export functioneel eindigt.

Context SlideRasterizer bewaart alle gerenderde PNGs in List<Uint8List> en verhoogt tegelijk de globale image-cache tot 1 GiB. PDF/PPTX-opbouw start pas nadat alle slides zijn gerasteriseerd. Impact Bij 1920×1080 is één ruwe capture circa 7,9 MiB. Honderden hoog-entropische slides kunnen meerdere gigabytes aan renders vasthouden, nog vóór PDF/ZIP-assemblage. Relevante code - lib/services/slide_rasterizer.dart:92 en 123 - lib/services/export_service.dart:477 en 527 Acceptatie - Rasteriseer en schrijf met een klein expliciet begrensd venster, of weiger vooraf op een onderbouwd totaalbudget. - Annuleren ruimt tussenresultaten en image-cache altijd op. - Een schaaltest met een groot deck bewaakt piekgeheugen/gelijktijdig levende renders, niet alleen dat export functioneel eindigt.

brenno added the

enhancement

accepted

labels

2026-08-01 08:26:06 +00:00

brenno referenced this issue from a commit

2026-08-01 20:13:38 +00:00

fix(export): weiger raster-export vooraf op een totaalbudget en ruim op (#1047)

brenno referenced this issue from a commit

2026-08-01 20:13:38 +00:00

chore(l10n): vertaal raster-export-budgetmelding (#1047)

brenno referenced this issue from a commit

2026-08-01 20:13:38 +00:00

refactor(export): splits rasterize onder de methodelengte + lijn cancel-contract (#1047)

brenno referenced this issue

2026-08-01 20:13:53 +00:00

perf(export): weiger raster-export vooraf op een totaalbudget en ruim op (#1047) #1091

brenno referenced this issue from a commit

2026-08-01 20:13:59 +00:00

Merge pull request 'perf(export): weiger raster-export vooraf op een totaalbudget en ruim op (#1047)' (#1091) from fix/export-raster-window-1047 into main

brenno commented

2026-08-01 20:14:07 +00:00

Author

Owner

Opgelost op main (PR #1091, merge in c4a2fad2). De raster-export weigert nu vóóraf op een onderbouwd totaalbudget (kMaxRasterExportBytes = 4 GiB) met een melding, en ruimt de image-cache plus tussenresultaten na afloop én bij annulering altijd op. Schaaltest bewijst weigering-zonder-render boven het budget. Kanttekening: een echt streamend venster vergt een grotere assemblage-refactor (pdf/archive houden het hele document in geheugen); het vooraf-budget is de tweede geaccepteerde optie uit het issue.

Opgelost op main (PR #1091, merge in c4a2fad2). De raster-export weigert nu vóóraf op een onderbouwd totaalbudget (kMaxRasterExportBytes = 4 GiB) met een melding, en ruimt de image-cache plus tussenresultaten na afloop én bij annulering altijd op. Schaaltest bewijst weigering-zonder-render boven het budget. Kanttekening: een echt streamend venster vergt een grotere assemblage-refactor (pdf/archive houden het hele document in geheugen); het vooraf-budget is de tweede geaccepteerde optie uit het issue.

brenno closed this issue

2026-08-01 20:14:07 +00:00