Skip to content

Blokkeer meterregel bij ongeldige start-HF - #27

Merged
streuper merged 1 commit into
devfrom
codex/v1.30.0-meterregel-start-hf
Jul 16, 2026
Merged

Blokkeer meterregel bij ongeldige start-HF#27
streuper merged 1 commit into
devfrom
codex/v1.30.0-meterregel-start-hf

Conversation

@streuper

Copy link
Copy Markdown
Collaborator

Wat is gewijzigd

De 6-, 9- en 12-meterregel worden alleen nog toegepast wanneer de inkomende herhalingsfactor bij het begin van de meterregelsessie exact 1,0 is.

Een vooraf gegroepeerde kandidaat-meterregelsessie met een afwijkende of onbetrouwbare start-HF wordt teruggebracht naar afzonderlijke chronologische duikeenheden. Deze worden vervolgens via de bestaande normale herhalings- of blokkaderoute beoordeeld.

Bij een actieve ketenblokkade worden geen kunstmatige waarden meer ingevuld voor HF, EDT, NDL of HG.

Waarom is dit nodig

De eerdere implementatie gaf bij een niet-vervallen herhalingsgroep alleen een waarschuwing, maar paste daarna alsnog de meterregel toe. Daarbij werd HF kunstmatig op 1 gezet en werd de duiktijd cumulatief als meterregel-EDT verwerkt.

Dit kon een ongeldige meterregelsessie als geldige meterregel presenteren.

Bron en besluitvorming

De meterregels vereisen dat de eerste duik van de meterregelsessie begint met HF 1,0.

Deze wijziging past geen nieuwe tabelwaarden, afrondingsregels of operationele limieten toe. De bestaande genormaliseerde uitkomst van getHF wordt exact met 1 vergeleken.

Veiligheidsimpact

Dit is een veiligheidswijziging.

Geraakt:

  • 6-meterregel;
  • 9-meterregel;
  • 12-meterregel;
  • start-HF;
  • normale herhalingsroute;
  • ketenblokkade;
  • EDT-, NDL- en HG-uitkomsten na blokkade.

Niet geraakt:

  • Airtabel 1;
  • tabel 4a;
  • tabel 4b;
  • DCIEM-tabelwaarden;
  • bronfixture;
  • bronfingerprints;
  • meterregellimieten;
  • gecombineerde-duikberekening;
  • overige oppervlakte-intervalregels.

Geraakte bestanden

  • index.html

    • controle van start-HF;
    • opsplitsing naar afzonderlijke chronologische eenheden;
    • defensieve controle in calcUnit;
    • uitsluiting uit meterregel-periodetelling;
    • gerichte regressietests.
  • CHECKSUMS.sha256

    • bijgewerkte checksum van index.html.

Tests en controles

Uitgevoerd en geslaagd:

  • JavaScript-syntaxcontrole;
  • 470/470 ingebouwde zelftests;
  • themakleuren-lint;
  • checksumcontrole;
  • git diff --check;
  • aanvullende randprobe voor een numeriek afgewezen start-HF met daaropvolgende ketenblokkade.

Gericht gecontroleerd:

  • geldige 6-, 9- en 12-meterregels met start-HF 1,0 blijven werken;
  • afgewezen meterregels krijgen geen kunstmatige HF 1;
  • afgewezen meterregels gebruiken geen cumulatieve meterregel-EDT;
  • ketengeblokkeerde vervolgduiken houden HF, EDT, NDL en HG leeg;
  • gewone eerste duiken blijven HF 1 krijgen;
  • een oppervlakte-interval boven 18 uur blijft resetten naar HF 1.

Rekenkern en bronfingerprint

  • DCIEM-tabelwaarden zijn niet gewijzigd.
  • Airtabel 1 is niet gewijzigd.
  • Tabel 4a en tabel 4b zijn niet gewijzigd.
  • De bronfixture is niet gewijzigd.
  • De bronfingerprints zijn niet gewijzigd.
  • Alleen de applicatiechecksum van index.html is bijgewerkt.

Privacy, opslag en uitwisseling

Geen wijzigingen aan:

  • localStorage;
  • import of export;
  • JSON, CSV, XLSX of OSOD;
  • persoonsgegevens;
  • Content Security Policy;
  • netwerkgedrag;
  • offline werking.

Compatibiliteit en migratie

Geen datamigratie nodig.

Bestaande opgeslagen duiken en exports blijven bruikbaar. Alleen de beoordeling van kandidaat-meterregelsessies met een ongeldige start-HF verandert.

Open punten en beperkingen

docs/VALIDATIE_TABELLEN.md beschrijft de oude situatie nog als een waarschuwing. Dit document is bewust niet gewijzigd omdat documentatie buiten de afgebakende reparatie viel. Dit wordt in een afzonderlijke documentatiewijziging verwerkt.

Deze wijziging lost niet de afzonderlijke problemen op rond:

  • dagdeelgroepering;
  • volledige chronologische meterregelsessies;
  • fasenregistratie;
  • handmatige DT;
  • luchtberekening;
  • OSOD-export;
  • volledige gecombineerde-duikberekening.

Controle voor de beheerder

Controleer vóór merge:

  • dat alle GitHub Actions groen zijn;
  • dat uitsluitend index.html en CHECKSUMS.sha256 zijn gewijzigd;
  • dat de DCIEM-tabellen en fingerprints ongewijzigd zijn;
  • dat de nieuwe meterregeltests inhoudelijk aansluiten op de bronvoorwaarde HF 1,0;
  • dat het openstaande documentatiepunt afzonderlijk wordt verwerkt.

@streuper
streuper changed the base branch from main to dev July 16, 2026 19:48
@streuper
streuper marked this pull request as ready for review July 16, 2026 20:09
@streuper
streuper merged commit 9eff6ae into dev Jul 16, 2026
2 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.

1 participant