Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign up[REVIEW]: ENZO: An Adaptive Mesh Refinement Code for Astrophysics (Version 2.6) #1636
Comments
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. @zingale, @rtfisher it looks like you're currently assigned to review 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:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
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.
danielskatz
commented
Aug 9, 2019
I had to update the repository URL on the top comment of this issue. (note to @arfon - whedon didn't use the URL in the pre-report issue, and this may also lead to a problem when we do the final acceptance.) |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Aug 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.
danielskatz
commented
Aug 9, 2019
@bwoshea - merging enzo-project/enzo-dev#93 will fix 2 small bib entries |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Aug 9, 2019
If you have any questions or problems, please let me know. |
whedon
assigned
rtfisher and
zingale
Aug 10, 2019
This comment has been minimized.
This comment has been minimized.
I am just beginning the review process. I find the documentation link (https://enzo-project.org/docs/2.6/) under https://github.com/enzo-project/enzo-dev returns a 404. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Aug 11, 2019
Now that this has been done, I will recompile - @bwoshea, you can do this too, just the way I am, with an instruction in whedon in a new comment (letting you know in case more changes are needed) |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Aug 11, 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.
regarding
There are 2 references to Enzo itself, referring to github. Should these be changed to an archived version of the code with a DOI? |
This comment has been minimized.
This comment has been minimized.
I'm done with my pass through the docs and have filed a number of issues, some which I would like to see addressed for the review (marked with I will try running some standard test problems next. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Aug 15, 2019
@zingale - thanks for all your comments so far. If possible, when you open an issue in https://github.com/enzo-project/enzo-dev, just mention this review thread (#1636) in that issue rather than posted that issue in this thread. This will insert a note here that will show if the issue is open or closed. |
This comment has been minimized.
This comment has been minimized.
oh, I see, I did it backwards. I'll fix that. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Aug 26, 2019
|
This comment has been minimized.
This comment has been minimized.
Just to clarify, I am still in the process of completing my review. @bwoshea indicates that the ENZO team is awaiting this completion prior to starting their responses. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Aug 26, 2019
That's ok, but JOSS reviews typically are more interactive. Maybe with a complex team like this, it makes sense to wait. |
This comment has been minimized.
This comment has been minimized.
bwoshea
commented
Aug 29, 2019
@danielskatz Sorry about that. Given the vigor of @zingale 's review and the distributed nature of the Enzo team, we would prefer to wait until the reviews are completed (or at least the first wave of feedback) until we proceed. If you insist on doing it otherwise I'm sure we can make that happen, though. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Aug 29, 2019
It’s fine to wait, as I said previously |
This comment has been minimized.
This comment has been minimized.
Ok, I've completed my initial review. It looks like I also identified some of the same issues as @zingale, though our final assessments of the JOSS requirements differ slightly. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Aug 29, 2019
This comment has been minimized.
This comment has been minimized.
bwoshea
commented
Sep 7, 2019
@danielskatz We're on it! @rtfisher and @zingale , thank you for all of your feedback! |
This comment has been minimized.
This comment has been minimized.
bwoshea
commented
Sep 12, 2019
@danielskatz @rtfisher @zingale The Enzo developers have been working on the issues you raised (at https://github.com/enzo-project/enzo-dev/issues), and we currently have a bunch of open pull requests to address them (at https://github.com/enzo-project/enzo-dev/pulls). Some of the issues have comments on them that directly address reviewer comments without a PR (e.g., enzo-project/enzo-dev#108), but mostly the changes in the PRs themselves should address your comments. Please let us know how you'd like to proceed at this point! |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Sep 12, 2019
@bwoshea - it's up to you, with the goal of getting the reviewers to agree that the submission meets the criteria. You could make all the changes, then ask the reviewers to check them, or ask reviewers for specific opinions on possible changes first. |
This comment has been minimized.
This comment has been minimized.
I would suggest that you refer to the PRs in the issues we raised and when we are satisfied, we will close the issues that we opened for the review, and that will be reflected here automatically. |
This comment has been minimized.
This comment has been minimized.
bwoshea
commented
Sep 12, 2019
@danielskatz -- duly noted, thank you! We'll engage the reviewers on the open PRs rather than waiting. That way we'll be able to respond more quickly. @zingale -- Great suggestion, thanks. I've done that now! We have a couple of outstanding issues that still need developer attention, but the bulk have been addressed. |
This comment has been minimized.
This comment has been minimized.
bwoshea
commented
Sep 13, 2019
•
@zingale @rtfisher just to give you a heads-up, we've addressed all of the issues you have raised but one, and that should be done in the next day or so. We've left all off the PRs open so that you can see what has been changed, and all PRs and issues are cross-referenced. We're ready for your feedback whenever you are available. Edit: we now (as of 2019-09-15) have PRs that address all issues raised! |
This comment has been minimized.
This comment has been minimized.
It looks like all of my issues have been addressed via PRs. Once those PRs are merged, I am happy with accepting. |
This comment has been minimized.
This comment has been minimized.
I am fine with accepting after merger as well. |
This comment has been minimized.
This comment has been minimized.
bwoshea
commented
Sep 16, 2019
@rtfisher to clarify, there are two open issues that you raised (enzo-project/enzo-dev#108 and enzo-project/enzo-dev#109 ) - would you mind making a quick note saying you're OK with what we've done for those? |
This comment has been minimized.
This comment has been minimized.
Thank you, @bwoshea. Sorry for the delayed response -- I was serving on a panel today. I concur that the issues are now resolved, and have closed both. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Sep 18, 2019
The next steps will be
|
whedon commentedAug 9, 2019
•
edited by rtfisher
Submitting author: @bwoshea (Brian O'Shea)
Repository: https://github.com/enzo-project/enzo-dev
Version: v2.6
Editor: @danielskatz
Reviewer: @zingale, @rtfisher
Archive: Pending
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
@zingale & @rtfisher, 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 @danielskatz know.
Review checklist for @zingale
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?Review checklist for @rtfisher
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?