Track tasks and feature requests
Join 36 million developers who use GitHub issues to help identify, assign, and keep track of the features and bug fixes your projects need.
Sign up for free See pricing for teams and enterprises[REVIEW]: Fireworks: Reproducible Machine Learning with PyTorch #1478
Comments
whedon
assigned
arokem
May 25, 2019
whedon
added
the
review
label
May 25, 2019
whedon
referenced this issue
May 25, 2019
Closed
[PRE REVIEW]: Fireworks: Reproducible Machine Learning with PyTorch #1422
This comment has been minimized.
This comment has been minimized.
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @dirmeier it looks like you're currently assigned as the reviewer for this paper If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/openjournals/joss-reviews) 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:
|
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
whedon
assigned
dirmeier
May 26, 2019
This comment has been minimized.
This comment has been minimized.
smk508
commented
May 31, 2019
Hi @dirmeier . Do you have any updates on this review or any questions? |
This comment has been minimized.
This comment has been minimized.
arokem
commented
Jun 13, 2019
Hello @dirmeier! Have you had a chance to take a look? |
This comment has been minimized.
This comment has been minimized.
Yeah, the review is in the repo's issue tracker.. |
This comment has been minimized.
This comment has been minimized.
smk508
commented
Jun 26, 2019
@whedon commands |
This comment has been minimized.
This comment has been minimized.
Here are some things you can ask me to do:
|
This comment has been minimized.
This comment has been minimized.
smk508
commented
Jun 26, 2019
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
PDF failed to compile for issue #1478 with the following error: /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-18b9a37e7f54/lib/whedon/orcid_validator.rb:12:in |
This comment has been minimized.
This comment has been minimized.
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
PDF failed to compile for issue #1478 with the following error: /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-85c572552d68/lib/whedon/orcid_validator.rb:12:in |
This comment has been minimized.
This comment has been minimized.
Hey @arokem , |
This comment has been minimized.
This comment has been minimized.
arokem
commented
Jul 3, 2019
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
PDF failed to compile for issue #1478 with the following error: /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-85c572552d68/lib/whedon/orcid_validator.rb:12:in |
This comment has been minimized.
This comment has been minimized.
arokem
commented
Jul 3, 2019
@openjournals/joss-eics : does anyone know what this error means? I am failing to debug this just by looking at the markdown. |
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Jul 3, 2019
I would try removing the empty |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Jul 3, 2019
something is wrong with an orcid |
This comment has been minimized.
This comment has been minimized.
arokem
commented
Jul 7, 2019
@smk508 : I implemented @kyleniemeyer's suggestion here: kellylab/Fireworks#75. If you merge that, we can take a look and see what remains to complete the review of this paper. |
This comment has been minimized.
This comment has been minimized.
smk508
commented
Jul 8, 2019
Hi @arokem , Sorry I was on vacation for the weekend so I didn't see this. I've merged the PR. |
This comment has been minimized.
This comment has been minimized.
smk508
commented
Jul 8, 2019
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
PDF failed to compile for issue #1478 with the following error: /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-85c572552d68/lib/whedon/author.rb:58:in |
This comment has been minimized.
This comment has been minimized.
smk508
commented
Jul 9, 2019
Ok let's see if that worked. @whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
smk508
commented
Jul 9, 2019
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
smk508
commented
Jul 9, 2019
ok it builds now |
This comment has been minimized.
This comment has been minimized.
arokem
commented
Jul 15, 2019
Sorry for the slowness here. One more comment: the figure caption contains the words "Figure 1". Please remove these, so that you don't get a duplication of these words in the PDF. Also, the reference to the Paszke paper seems partial. Is there more information? |
This comment has been minimized.
This comment has been minimized.
smk508
commented
Jul 16, 2019
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
smk508
commented
Jul 16, 2019
Ok I've updated the figure caption. The Paszke reference was meant to be a reference to the Pytorch library itself and its built-in data processing modules when I was comparing them to Fireworks. |
This comment has been minimized.
This comment has been minimized.
arokem
commented
Jul 16, 2019
Gotcha. Looks like it's missing a couple of words, though. See: https://github.com/pytorch/pytorch/blob/master/CITATION |
This comment has been minimized.
This comment has been minimized.
smk508
commented
Jul 16, 2019
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
smk508
commented
Jul 16, 2019
Oh sorry. I misunderstood what you were saying before! I fixed the typo in the citations now. |
This comment has been minimized.
This comment has been minimized.
arokem
commented
Jul 16, 2019
@whedon check references |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
arokem
commented
Jul 16, 2019
I think that Whedon is checking all the references in your bib file. I believe everything is in order here. |
This comment has been minimized.
This comment has been minimized.
arokem
commented
Jul 16, 2019
@smk508 : could you please create an archive of the software (e.g., using https://zenodo.org) and post the link + DOI here? I apologize in advance for the additional delay, but I am heading out on vacation and will be back heree next week to move this ahead. |
This comment has been minimized.
This comment has been minimized.
smk508
commented
Jul 17, 2019
Ok that sounds good. I've made an archive on zenodo by following thir instructions and added a badge to the README. |
This comment has been minimized.
This comment has been minimized.
arokem
commented
Jul 20, 2019
@whedon set 10.5281/zenodo.3339548 as archive |
This comment has been minimized.
This comment has been minimized.
OK. 10.5281/zenodo.3339548 is the archive. |
This comment has been minimized.
This comment has been minimized.
arokem
commented
Jul 20, 2019
@openjournals/joss-eics : I believe this paper is ready for your final review |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Jul 20, 2019
Is version 0.3.0 correct? It seems like perhaps this should be 0.3.2? |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Jul 20, 2019
Also, I don't think Kubeflow is a journal, so in paper.bib: |
This comment has been minimized.
This comment has been minimized.
smk508
commented
Jul 20, 2019
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
smk508
commented
Jul 20, 2019
Ok I've updated the kubeflow citation in paper.bib. The current version is 0.3.2, yes. I don't know how to change the top post in this thread to say 0.3.2. |
whedon commentedMay 25, 2019
•
edited
Submitting author: @smk508 (Saad Khan)
Repository: https://github.com/kellylab/Fireworks
Version: 0.3.0
Editor: @arokem
Reviewer: @dirmeier
Archive: 10.5281/zenodo.3339548
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
@dirmeier, 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://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @arokem know.
Review checklist for @dirmeier
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?