Skip to content

Kans elementen met een PoF van NaN resulteert in crash#104

Merged
chrispijo merged 2 commits into
alphafrom
bug_pof_of_nan
Jul 7, 2026
Merged

Kans elementen met een PoF van NaN resulteert in crash#104
chrispijo merged 2 commits into
alphafrom
bug_pof_of_nan

Conversation

@chrispijo

@chrispijo chrispijo commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

Bijgevoegd bestand resulteerde in NaN PoF-waarden in het Kans-element. Dit volgt hoogstwaarschijnlijk uit de invoer die nog in zijn iteraties zit. De volgende verbeteringen zijn aangebracht:

  • Acceptatie NaN-waarde in KansElement en UittredepuntElement.
  • In de visualisatie BetaMap werden NaN-resultaten niet geaccepteerd. Nu worden deze er uitgefilterd.
  • Geleidelijke opschoning van code: voornamelijk regellengte van 80 naar 120 aanpassen.

Bij het bespreken met Sander kwamen we tot de conclusie dat het KansElement niet gegenereerd moet worden bij een NaN-resultaat. Omdat de gebruiker verder moet, gaan we eerst releasen en later verder kijken. Als actielijst, plus nog een extra actie:

  • KansElement moet niet gegenereerd worden bij NaN-resultaat.
  • In BetaMap visualiseren welke uittredepunten NaN-waarden zijn.
  • Er ontstaat door de NaN-waarde nu een RuntimeWarning. Zie onderstaand. Dit is tijdens het exporteren van het archief.
\GeoProb-Pipe\.venv\Lib\site-packages\scipy\stats\_distn_infrastructure.py:2334: RuntimeWarning: invalid value encountered in multiply
  lower_bound = _a * scale + loc

@chrispijo chrispijo added the bug Something isn't working label Jul 7, 2026
@chrispijo chrispijo merged commit 45e8320 into alpha Jul 7, 2026
1 check passed
@chrispijo chrispijo deleted the bug_pof_of_nan branch July 7, 2026 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant