Repository navigation
minor generalizations - #2161
affeldt-aist wants to merge 3 commits into
Conversation
hoheinzollern
left a comment
There was a problem hiding this comment.
I understand the need to generalize expectation to extended real-valued random variables, but shouldn't there be notations for handling the real case, like for the integral? The composition EFin \o f could be automatically handled by the scope.
|
This generalization is fine with me. I added some minor comments. In step with this generalization, should I change the codomain of This is also the case with |
Note that in the case of the integral there is a new definition ( |
gen finite_form
|
Thanks for the comments. Regarding the type of |
2b31396 to
95cabd1
Compare
|
by the way @Yosuke-Ito-345 @hoheinzollern you might be interested in providing input to #2116 in which @lyonel2017 put much effort |
Motivation for this change
generalize expectation to extended real-valued RVs
gen finite_form
motivated by PR #2116
Checklist
CHANGELOG_UNRELEASED.mdReference: How to document
Merge policy
As a rule of thumb:
all compile are likely to be merged as they are into master.
Reminder to reviewers