-
Notifications
You must be signed in to change notification settings - Fork 53
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BUG] bugfixes to QPD distributions - QPD_U
, QPD_S
#194
Conversation
… are resolved"
CyclicBoosting
and QPD tests once #189 failures are resolvedCyclicBoosting
and QPD tests once #190 failures are resolved
CyclicBoosting
and QPD tests once #190 failures are resolved
@setoguchi-naoki, @FelixWick, I have been trying to fix the open bug after re-enabling the tests, it seems like there are some issues with the internal indexing. I'm not sure what format of |
alpha: float, |
I mean, internally. You do this: |
Thanks, this seems to have helped. Now, It should not substitute the bounds, because they will always be nan, so where is that coming from? |
QPD_U
, QPD_S
FYI, it seems the bugs we were targeting are now resolved. I have opened a bug issue here, to track that separately: #257 |
Re-enables tests for
QPD_U
,QPD_S
skipped in #193 and attempts to fix failures in QPD distributions.it appears that #190 failures are resolved - fixes #190.
Further
QPD_B
seems to fail at mean and var calculations, this is a separate bug and raised as #257. As a mechanical consequence,QPD_B
is left skipped in the tests.