-
Notifications
You must be signed in to change notification settings - Fork 18
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
xarray wrapper to draw auto-regression samples #322
Conversation
return size, coord_dict | ||
|
||
|
||
def _draw_auto_regression_uncorrelated( |
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.
@yquilcaille can you read the docstrings and let me know if this makes sense?
I added a leading underscore to the name as I am not yet sure I'll expose the functions as is, but that can be changed.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #322 +/- ##
==========================================
+ Coverage 87.16% 87.57% +0.41%
==========================================
Files 38 38
Lines 1620 1674 +54
==========================================
+ Hits 1412 1466 +54
Misses 208 208
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
isort . && black . && flake8
CHANGELOG.rst
A lot is still missing (tests, docs). But as a first draft.