-
Notifications
You must be signed in to change notification settings - Fork 12
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
fix simulate #479
fix simulate #479
Conversation
Codecov Report
@@ Coverage Diff @@
## master #479 +/- ##
==========================================
- Coverage 78.03% 78.00% -0.04%
==========================================
Files 23 23
Lines 2295 2300 +5
Branches 544 546 +2
==========================================
+ Hits 1791 1794 +3
- Misses 369 370 +1
- Partials 135 136 +1
Continue to review full report at Codecov.
|
Kudos, SonarCloud Quality Gate passed!
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Co-authored-by: Daniel Weindl <dweindl@users.noreply.github.com>
Kudos, SonarCloud Quality Gate passed!
|
Default noise distribution wasn't handled in some cases Add option to ensure synthetic data is non-negative (can become negative after noise is added) * fix default noise distribution * switch to pandas isna * Return negative values by default. * change non-negative argument to zero-bounded * docs, set random seed Co-authored-by: dilpath <59329744+dilpath@users.noreply.github.com> Co-authored-by: Daniel Weindl <dweindl@users.noreply.github.com>
TODOs
ensure synthetic data file rows are ordered as in the measurements file