Closed andrewrosemberg closed 1 year ago
:exclamation: No coverage uploaded for pull request base (
master@ad38b3f
). Click here to learn what that means. Patch has no changes to coverable lines.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Summary
Add tests and lightly refactor the code to be usable. Much of the changes proposed here were developed by Copilot and Chatgpt under my supervision.
Changes
num_samples
) that keeps track of how many samples the user wants to use in sample-based formulations from insideConditionalExpectedReturn
toDeterministicSamples
.