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]: ShOpt.jl: A Julia Package for Empirical Point Spread Function Characterization of JWST NIRCam Data #6144
Comments
Hello humans, I'm @editorialbot, a robot that can help you with some common editorial tasks. 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:
|
|
Wordcount for |
|
Review checklist for @kevinmichaConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Hi @EdwardBerman et al., I have been reviewing the I have opened two issues in your repository so far; please let me know when they are fixed so I can continue to check the boxes: |
Review checklist for @jpierel14Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Hi, @EdwardBerman et al. have solved the issues I opened via the Dec 23, 2023 and Jan 4, 2024 commits. @ivastar I have ticked all the boxes and am happy to recommend |
@kevinmicha thank you for the expeditious review! Pinging @jpierel14 with a reminder to please complete the outstanding items in the checklist. Let me know if I can be of assistance. |
@editorialbot add @aymgal as reviewer |
@aymgal added to the reviewers list! |
@EdwardBerman, adding one more reviewer as @jpierel14 reported issues with the functionality testing due to IT issues at his institute. @aymgal, please only do the functionality testing part of the checklist. Let me know if there are questions. |
@ivastar Sorry can you point me to where the most recent version of the paper is? I read the version linked above, but then noticed in here that the paper had been updated. I see in the repo, the pdf has not been updated but the markdown version of the paper has, and does not match the pdf linked above by the bot. I have read the paper though and am ready to give comments, just want to make sure I'm on the most recent version! |
@editorialbot generate pdf |
Does this update the PDF for you? Otherwise I'll push an updated PDF to the repository. |
@EdwardBerman ah yep that one is updated, thank you! |
@editorialbot generate pdf |
Regenerated pdf to fix one last typo |
@editorialbot generate pdf |
@editorialbot check references |
Checking the BibTeX entries failed with the following error:
|
@editorialbot generate pdf |
@editorialbot check references |
Checking the BibTeX entries failed with the following error:
|
@editorialbot generate pdf |
@editorialbot check references |
|
@editorialbot check references |
|
@editorialbot check references |
|
@ivastar I've updated all references to published versions and tracked down all of the DOI's except for one. I can't seem to find the one for PSFex anywhere, not on NASA ADS, google scholar, or any publishing site. I'm not sure it exists? Can you suggest to me a next step. Beyond that, I have one more writing change I'd like to make per your editorial comments, and then I will have addressed everything. Thanks! |
Doh. This is the only thing I can find for PSFex: https://ui.adsabs.harvard.edu/abs/2013ascl.soft01001B/exportcitation |
@editorialbot check references |
@editorialbot generate pdf |
@editorialbot generate preprint |
|
📄 Preprint file created: Find it here in the Artifacts list 📄 |
@ivastar I've finished going through your editorial comments and tracking down the correct DOI's. We've received our referee report for our AAS companion paper in the last couple days, and are making our way through the necessary revisions at this time. Let me know if you need anything else on our end, otherwise, I think all that is left is for my coauthors and I to work through the AAS review process. Thanks again for your help! |
Excellent! Do you think that the code will change in response to the AAS comments? If not, we can may be move ahead with the post-review checklist: |
For the most part any code changes should just be minor edits. I don't anticipate any major changes, although, I'll have to dig deeper into these comments before I can say this for certain. |
@EdwardBerman following up on this review. Please let me know if there are any developments. |
Hi @ivastar thank you for checking in. I've made a handful of small edits which are now pushed. I'm hoping to finish things up soon for resubmitting the AAS companion paper. I will keep you updated when that happens and any major developments along the way! |
Submitting author: @EdwardBerman (Edward Berman)
Repository: https://github.com/EdwardBerman/shopt
Branch with paper.md (empty if default branch):
Version: v2.0.1
Editor: @ivastar
Reviewers: @kevinmicha, @jpierel14, @aymgal
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
@kevinmicha & @jpierel14, your review will be checklist based. Each of you will have a separate checklist that you should update when carrying out your review.
First of all you need to run this command in a separate comment to create the checklist:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @ivastar know.
✨ Please start on your review when you are able, and be sure to complete your review in the next six weeks, at the very latest ✨
Checklists
📝 Checklist for @kevinmicha
📝 Checklist for @jpierel14
The text was updated successfully, but these errors were encountered: