Skip to content

Fix assertion error in normal distribution#68

Open
Butakus wants to merge 3 commits into
EasyNavigation:rollingfrom
Butakus:fix_67_normal_dist
Open

Fix assertion error in normal distribution#68
Butakus wants to merge 3 commits into
EasyNavigation:rollingfrom
Butakus:fix_67_normal_dist

Conversation

@Butakus

@Butakus Butakus commented Jul 7, 2026

Copy link
Copy Markdown
Member

Hi,

This is a follow-up of #66, that fixes #67.
I submitted a new PR in case we prefer to fix the CI first, and then fix the assertion error in a different way than the one I propose here:

I added a check for non-positive standard deviations, which raises an exception when initializing the plugin.
Some tests were setting the std to 0.0. I just moved those values to something really small (1e-12) which is roughly equivalent.

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.

Assertion error using std params with value 0.0

1 participant