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]: pyRTC: An open-source Python solution for kHz real-time control of adaptive optics systems #6466
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:
|
|
Software report:
Commit count by author:
|
Paper file info: 📄 Wordcount for ✅ The paper includes a |
License info: 🟡 License found: |
@joseph-long, @joao-aveiro, @sefffal – This is the review thread for the paper. All of our communications will happen here from now on. Please read the "Reviewer instructions & questions" in the first comment above. Please create your checklist typing:
As you go over the submission, please check any items that you feel have been satisfied. There are also links to the JOSS reviewer guidelines. 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 make a start well 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. |
@jacotay7 – is this JOSS submission/paper is a joint publication with AAS Journals? |
@arfon Nope, not a joint publication. |
Review checklist for @joao-aveiroConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Review checklist for @sefffalConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Review checklist for @joseph-longConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Hi @jacotay7, I have a few initial questions & comments. Second, I notice that the documentation is a little sparse so far. There is a great quick-start notebook that would tell me how to set up a basic system. But a typical AO system would be expected to need a fair amount of application-specific customization. As it stands, I'm not sure the example notebook would provide enough information for me to know how to create a multi-stage pipeline, add new wavefront sensors, or support for a DM, for instance. Would you consider writing an additional page explaining the architecture of the software? If this documentation is already written and I have overlooked it, please disregard! |
Yeah, I know that there is already a pyRTC on pip, I think I will change the name once I decide things are far enough along to be put on pip, and I will switch to the name pyAORTC or similar. For now, I think all users will be cloning the repo individually and I think the likelihood of conflict with the exiting repository is very small.
Sorry about this, I am in the process of getting a guide written with details on the implementation. I have been keeping a full config and RTC example for my lab system in the main branch to compensate for the lack of this guide. I will update you when this guide is avaliable. On a more general note, this package is very hard to test since it is a hardware interfacing code, which is why I put together a simulation example in the first place. The simulation examples cannot really fully capture the utility of the program the same way a hardware implementation can, without some imagination on the part of the user. If anyone has any suggestions for how to bridge that gap for the review let me know. |
Submitting author: @jacotay7 (Jacob Taylor)
Repository: https://github.com/jacotay7/pyRTC
Branch with paper.md (empty if default branch):
Version: v1.0.0
Editor: @arfon
Reviewers: @joseph-long, @joao-aveiro, @sefffal
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
@joseph-long & @joao-aveiro & @sefffal, 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 @arfon 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 @joao-aveiro
📝 Checklist for @sefffal
📝 Checklist for @joseph-long
The text was updated successfully, but these errors were encountered: