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. |
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.
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
|
This comment has been minimized.
This comment has been minimized.
bwoshea
commented
Sep 19, 2019
@danielskatz we have made the remaining changes! |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Sep 19, 2019
This comment has been minimized.
This comment has been minimized.
I'm happy to accept. I've check all my boxes. Nice job @bwoshea and the Enzo team. |
This comment has been minimized.
This comment has been minimized.
I accept as well, and second Michael's congratulations to @bwoshea and all of the many Enzo developers! |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Sep 19, 2019
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Sep 19, 2019
•
|
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Sep 19, 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
Sep 19, 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.
danielskatz
commented
Sep 22, 2019
deposit the repository in an archive (e.g. zenodo) and report the DOI here |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Sep 29, 2019
|
This comment has been minimized.
This comment has been minimized.
bwoshea
commented
Oct 2, 2019
•
@danielskatz sorry for the delay. We now have a DOI: 10.5281/zenodo.3469922: The lates version number is Enzo v2.6.1. We are verifying the paper text and references right now. The one issue I can see with the paper is the overlapping text in the bottom-left corner of the first page, where the 'LICENSE' block overlaps the footer: I anticipate that I will get back to you by no later than tomorrow finalizing the paper text and references. |
This comment has been minimized.
This comment has been minimized.
@bwoshea @danielskatz - this is an issue with our LaTeX template not handling long author lists very well. @danielskatz - feel free to accept this paper with Whedon here but I'll need to manually update the PDF afterwards to fix the layout issues. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Oct 2, 2019
@whedon set 10.5281/zenodo.3469922 as archive |
This comment has been minimized.
This comment has been minimized.
OK. 10.5281/zenodo.3469922 is the archive. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Oct 2, 2019
@whedon set v2.6.1 as version |
This comment has been minimized.
This comment has been minimized.
OK. v2.6.1 is the version. |
whedon commentedAug 9, 2019
•
edited
Submitting author: @bwoshea (Brian O'Shea)
Repository: https://github.com/enzo-project/enzo-dev
Version: v2.6.1
Editor: @danielskatz
Reviewer: @zingale, @rtfisher
Archive: 10.5281/zenodo.3469922
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?