-
Notifications
You must be signed in to change notification settings - Fork 9
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
Parallel io additions #83
Parallel io additions #83
Conversation
…es obj as return value for later
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.
first round of review
src/satellites.jl
Outdated
end | ||
|
||
# Statically split range [1,N] into equal sized chunks for np processors | ||
function splitrange(N::Int, np::Int) |
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.
can you add a more extensive documentation here?
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.
I would appreciate if you could do this, don't know exactly what you did
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.
I didn't write the splitrange
function. Can you combine it with the splitting
function?
Codecov Report
@@ Coverage Diff @@
## develop #83 +/- ##
==========================================
+ Coverage 89.41% 89.72% +0.3%
==========================================
Files 8 8
Lines 586 613 +27
==========================================
+ Hits 524 550 +26
- Misses 62 63 +1
Continue to review full report at Codecov.
|
Thanks @oHunewald 👍 |
No description provided.