-
-
Notifications
You must be signed in to change notification settings - Fork 36
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]: ScatteringOptics.jl: An Interstellar Scattering Framework in the Julia Programming Language #6354
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 |
|
@Edenhofer, @tomkimpson — This is the review thread for the paper. All of our correspondence will happen here from now on. Thanks again for agreeing to participate! 👉 Please read the "Reviewer instructions & questions" in the first comment above, and generate your checklists by commenting The JOSS review is different from most other journals. Our goal is to work with the authors to help them meet our criteria instead of merely passing judgment on the submission. As such, the reviewers are encouraged to submit issues and pull requests on the software repository. When doing so, please mention We aim for the review process to be completed within about 4-6 weeks but please try to make a start ahead of this as JOSS reviews are by their nature iterative and any early feedback you may be able to provide to the author will be very helpful in meeting this schedule. Please get your review started as soon as possible! |
Review checklist for @EdenhoferConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
I would like to request a COI waiver for my review of ScatteringOptics.jl (#6354) regarding Paul Tiede, who is a co-author of said paper. I've been at the Center for Astrophysics | Harvard & Smithsonian, Paul's employer, for about half a year in 2023 and have had two extended conversations with Paul about Gaussian processes, a subject unrelated to the submission. Since we work on very different topics in astrophysics, have never collaborated, and haven't interacted regularly, I believe I can evaluate the submission impartially. |
PaperThe paper is well written, summarizes the need for the software well, and focuses on the essential mathematics. However, as a non-expert in the field of radio astronomy, I think that the mathematics discussed should be more closely tied to what ScatteringOptics.jl provides. In particular, it is not clear to me how the equation after l70 and the equation after l74 are related. I probably am missing something, but I thought the scattering is performed in image space and only Minor notes
|
@Edenhofer — Thanks for bringing this up! Given the weak nature of this potential COI, I'm happy for you to continue with the review, having noted this context. @annatartaglia — If you or any of your co-authors have any concerns about this at all please reach out to me here or over email (my address should be easy to find!). Thanks all!! |
Review checklist for @tomkimpsonConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
The package is useful and the accompanying paper is well-written. It was nice to learn about the use of Julia by the EHT collaboration. I have some general comments below. I will review the code itself separately and open any issues as needed.
|
Apologies for the delay in my response–I have been busy with graduate admissions visits the past few weeks. We would like to thank the referees (@Edenhofer @tomkimpson) for their constructive comments. We are in the process of carefully going through each review and revising the paper and repository accordingly. I will respond to individual comments as we revise. Thanks for your patience! |
@annatartaglia — Thanks for your previous update! I wanted to check in here to make sure that this is still on your radar. Let us know how things are going. Thanks! |
@annatartaglia — Any updates here? Please let us know what your timeline looks like for the next steps. If I don't hear from you in the next 2 weeks, I'll assume that this review has been abandoned, and reject the submission, so just let me know how things are going ASAP! |
Submitting author: @annatartaglia (Anna Tartaglia)
Repository: https://github.com/EHTJulia/ScatteringOptics.jl
Branch with paper.md (empty if default branch):
Version: v0.1.2
Editor: @dfm
Reviewers: @Edenhofer, @tomkimpson
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
@Edenhofer & @tomkimpson, 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 @dfm 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 @Edenhofer
📝 Checklist for @tomkimpson
The text was updated successfully, but these errors were encountered: