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
[REVIEW]: Basic Threading Examples in JuliaLang v1.3 #54
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @ArchRobison, @wsmoses it looks like you're currently assigned to review this paper 🎉. ⭐ Important ⭐ If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/JuliaCon/proceedings-review) repository. As a reviewer, you're probably currently watching this repository which means for GitHub's default behaviour you will receive notifications (emails) for all reviews 😿 To fix this do the following two things:
For a list of things I can do to help you, just type:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
|
PDF failed to compile for issue #54 with the following error: /app/vendor/ruby-2.4.4/lib/ruby/2.4.0/find.rb:43:in |
X-ref #49 (comment) |
@whedon generate pdf |
|
@whedon commands |
Here are some things you can ask me to do:
|
@whedon generate pdf |
|
@whedon generate pdf |
|
Is there anything else I need to do? I think I've adopted and incorporated all of Arch's feedback and made it better. |
Am I supposed to bump this? I'm not really sure what its status is or who it's waiting on. |
No, that's implied in the |
Ah that makes sense, in any case here's a more full review/set of comments: This paper presents a mechanism of embedding dynamic task-based parallelism into the Julia language, allowing Julia programs to benefit from parallelism as distinct from existing Julia mechanisms from concurrency. The core of this contribution is a new scheduler with frontend modifications that allow several front-end improvements to make it easier to use this model. The authors also describe several changes to language constructs needed to integrate their framework with arbitrary code, including a demonstration of using external libraries with their framework. The authors present several demonstrations of programs (merge sort, number sieve, and a prefix scan), that could benefit from the task-parallelism that they now permit. The additional syntax, while demonstrated in code blocks, isn't explained well for new users hoping to take advantage of it (e.g. no description of @sync/fetch, ambiguity about the guarantees provided by @Spawn). The authors allude to several limitations in existing Julia threading primitives. There isn't a good description of why the existing threading primitives couldn't be used on the demonstration codes (though I can see why this would be difficult). It would also be useful to do a performance comparison between the new runtime and old systems on some mutually compatible code. This comparison, along with perhaps showing the serial runtime [e.g. without parallel constructs rather than 1-core with parallel constructs] would be helpful to demonstrate that the new runtime doesn't provide significant overhead. It would also be desirable to provide more quantifiable justification of their performance on FFTW than simply claiming it to be competitive (which is somewhat unclear). The authors should also include additional reference to related work in the field (e.g. existing parallel runtimes, including those outside of Julia) and describe the similarities and differences between prior art and the runtime described in this paper. As a minor point (and I'm not the most familiar with the types of papers in this venue so take this with a grain of salt): some of the wording e.g. "but on to even more fun stuff..." could be tightened up for a professional audience. Overall this paper represents a significant contribution to the Julia community in improving the experience of writing high performance parallel code. There are some additional changes to the writeup that would aid in fully demonstrating this contribution. |
@whedon check references |
|
@whedon generate pdf |
I don't have an archival DOI, but don't think it'll be useful for this. I think Kiran and I have improved the paper based on the reviewer's comments. |
What's next for us? |
@whedon commands |
Here are some things you can ask me to do:
|
@whedon accept |
No archive DOI set. Exiting... |
Can you create one with the example code being used? |
Reminder set for @vtjnash in 1 week |
I'm sorry @vtjnash, I'm afraid I can't do that. That's something only editors are allowed to do. |
@whedon set 10.5281/zenodo.4480494 as archive |
OK. 10.5281/zenodo.4480494 is the archive. |
@whedon generate pdf |
@whedon check references |
|
@whedon accept |
|
|
👋 @JuliaCon/jcon-eics, this paper is ready to be accepted and published. Check final proof 👉 JuliaCon/proceedings-papers#37 If the paper PDF and Crossref deposit XML look good in JuliaCon/proceedings-papers#37, then you can now move forward with accepting the submission by compiling again with the flag
|
@whedon accept deposit=true |
|
🚨🚨🚨 THIS IS NOT A DRILL, YOU HAVE JUST ACCEPTED A PAPER INTO JCON! 🚨🚨🚨 Here's what you must now do:
Any issues? Notify your editorial technical team... |
🎉🎉🎉 Congratulations on your paper acceptance! 🎉🎉🎉 If you would like to include a link to your paper from your README use the following code snippets:
This is how it will look in your documentation: We need your help! JuliaCon Proceedings is a community-run journal and relies upon volunteer effort. If you'd like to support us please consider doing either one (or both) of the the following:
|
Submitting author: @vtjnash (Jameson Nash)
Repository: https://github.com/vtjnash/JuliaCon2019Threading
Branch with paper.md (empty if default branch):
Version:
Editor: @vchuravy
Reviewers: @ArchRobison, @wsmoses
Archive:
Status
Status badge code:
Reviewers and authors:
Please avoid lengthy details of difficulties in the review thread. Instead, please create a new issue in the target repository and link to those issues (especially acceptance-blockers) by leaving comments in the review thread below. (For completists: if the target issue tracker is also on GitHub, linking the review thread in the issue or vice versa will create corresponding breadcrumb trails in the link target.)
Reviewer instructions & questions
@ArchRobison & @wsmoses , please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:
The reviewer guidelines are available here: https://proceedings.juliacon.org/guide/reviewers. Any questions/concerns please let @vchuravy know.
✨ Please try and complete your review in the next two weeks ✨
Review checklist for @ArchRobison
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Paper format
paper.tex
file include a list of authors with their affiliations?Content
Review checklist for @wsmoses
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Paper format
paper.tex
file include a list of authors with their affiliations?Content
The text was updated successfully, but these errors were encountered: