Allow unary negation to work with all distributions #96
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Fixed #95. I added a test showing that
-dist_wrap('norm')
and-dist_student_t(3)
fail before the fix and pass afterwards.Not sure if this is the optimal approach, but I just copy pasted the part from
Ops.dist_normal
that handled the unary negation to the code forOps.dist_default
andOps.dist_transformed