Skip to content

Revert regressive vector PDF zoom experiment - #321

Draft
DutchSailor wants to merge 1 commit into
mainfrom
codex/research-improvement-vector
Draft

Revert regressive vector PDF zoom experiment#321
DutchSailor wants to merge 1 commit into
mainfrom
codex/research-improvement-vector

Conversation

@DutchSailor

Copy link
Copy Markdown
Collaborator

Samenvatting

Draait de volledige vector-PDF-performance-experimentreeks uit de inmiddels gemergede PR #320 terug nadat praktijkgebruik met MV-03 een duidelijke vertraging liet zien. De productcode keert terug naar de toestand vóór het onderzoek (5d405c32).

Oorzaak van de regressie

De onderzoeksbenchmark stelde fysieke HiDPI-scherpte gelijk aan interactief gereed zijn. Daardoor werd op 150% niet langer een lichte tegel van 923×896 (schaal 1,5; circa 3,3 MB) gebruikt, maar een coverage-tegel van 2769×2688 (schaal 4,5; circa 29,8 MB).

Dat betekende negen keer zoveel pixels per zichtregio. Daarnaast kon speculatieve coverage-voorwarming op dezelfde worker voor echte zoom- en panverzoeken terechtkomen. Alternatieve combinaties van CSS-LOD, voorwarming en worker-affinity verschoven de wachttijd of verhoogden het geheugen tot circa 5,15 GB, maar leverden geen betrouwbare praktijkwinst op.

Wat deze PR doet

  • herstelt het oorspronkelijke lichte tegelpad;
  • verwijdert de HiDPI-verplichting uit het onderzochte zoompad;
  • verwijdert de coverage-prewarm- en cross-bucket-experimenten;
  • herstelt de oorspronkelijke workerverdeling en zoom-freeze;
  • verwijdert de bijbehorende benchmark/specs en het inmiddels weerlegde resultatenrapport.

Herstelmeting MV-03

Op het herstelde productpad:

  • openen: 8,9 s;
  • eerste 150%-tegel: 4,05 s;
  • daarna 150%/200%/300%-sprongen: mediaan 30,9 ms;
  • maximum warme zoomstap: 61,8 ms;
  • tegelgrootte: 923×896 in plaats van 2769×2688.

De eerste tegel blijft afhankelijk van de complexiteit van de PDF, maar verdere zoomstappen gebruiken opnieuw direct de lichte gecachte tegel.

Validatie

  • npm run test:unit: 66/66 geslaagd;
  • productie-frontendbuild: geslaagd;
  • live dev-build getest met MV-03;
  • volledige researchreeks in één traceerbare revertcommit teruggedraaid.

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.

1 participant