-
-
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]: normflows: A PyTorch Package for Normalizing Flows #5361
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 |
👋🏼 @VincentStimper, @matejgrcic, @kazewong, this is the review thread for the paper. All of our communications will happen here from now on. As a reviewer, the first step is to create a checklist for your review by entering
as the top of a new comment in this thread. There are additional guidelines in the message at the start of this issue. Please don't hesitate to ping me (@mstimberg) if you have any questions/concerns. |
|
Review checklist for @kazewongConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Review checklist for @matejgrcicConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
👋 @kazewong, @matejgrcic no rush for performing the actual review of course, but just checking in to see whether there are any questions on your side or anything else I can do to help you with the review? Please be reminded that JOSS reviews can be iterative, i.e. feel free to open issues or comment here as soon as you come across concerns – there is no need to gather everything into a single review text as commonly done for traditional journals. In the same vein, please feel free to tick boxes in your checklist whenever you are sure about them (the two in the beginning – conflict of interest and code of conduct – would be good first candidates 😉 ). Thanks for your time 🙏 ! |
Hi it's me again 😊 @kazewong, @matejgrcic, how are things going from your side, anything I can do to help? Please note that at JOSS we strive to complete a first round of reviews within 6 weeks, and this review issue has been opened about 4 weeks ago. Could you give me some estimate when you think will be able to perform the review? Thanks a lot, and as always, don't hesitate to ask in case anything about the process is unclear 🙏 PS: I will be travelling for the next two weeks, so my replies might take a bit longer than usual. But as I said earlier, reviews here can be iterative, and authors and reviewers are free to discuss and fix issues without my direct involvement 😉 |
@mstimberg I just came back from my work travel. There is a submission deadline this coming Friday, but after that I should be able to start putting in review issues in the repo. That should be aligned with the 6 weeks deadline. |
@kazewong Thanks for the feedback. If you could finish the review in the week after the submission deadline, that would be perfect! I'll set you up a reminder here, given that I might not be available myself to poke you 😊 |
@editorialbot remind @kazewong in 10 days |
Reminder set for @kazewong in 10 days |
👋 @kazewong, please update us on how your review is going (this is an automated reminder). |
👋 Hi @kazewong, how are things going? Could you please let us know whether you found time to have a look at the software and when you will be able to finish your review 🙏 ? |
👋 Hi @matejgrcic I've seen that you started your review and checked off a few items in the checklist. Could you let us know when you will be able to finish your review 🙏 ? |
@mstimberg I have some comments for the draft which I am going to start putting them in by the end of today. |
Hi @mstimberg, I should finish the review this week. |
@kazewong and @matejgrcic, thanks for your feedback, looking forward to your reviews/comments 👍 |
Hi @VincentStimper, thanks for the changes and the release, we are almost there! I noticed that the Zenodo record lists the licence "Other" instead of MIT, could you change this manually in the archive metadata (no need for a new release)? |
@editorialbot set version v1.7.0 |
I'm sorry human, I don't understand that. You can see what commands I support by typing:
|
@editorialbot set v1.7.0 as version |
Done! version is now v1.7.0 |
@editorialbot set 10.5281/zenodo.8027667 as archive |
Done! archive is now 10.5281/zenodo.8027667 |
@editorialbot generate pdf |
Oh, and I'm sorry, but I overlooked it before: would you mind also manually editing the author list in the archive so that it matches the author list of the paper? 🙏 |
Hi @mstimberg, I changed the author list and the license. Best regards, |
Thanks for the changes. Apologies for the nitpicking, but it seems that the Zenodo archive incorrectly associates Bernhard Schölkopf's ORCID with José Miguel Hernández-Lobato. As soon as this is fixed we will be ready to go. |
Hi @mstimberg, thanks for spotting this. Now it's changed. Best regards, |
Great, all looks good from my side! I will now recommend the paper for submission, and an editor-in-chief will launch the final steps for the publication process. |
@editorialbot recommend-accept |
|
|
👋 @openjournals/dsais-eics, this paper is ready to be accepted and published. Check final proof 👉📄 Download article If the paper PDF and the deposit XML files look good in openjournals/joss-papers#4300, then you can now move forward with accepting the submission by compiling again with the command |
I will process this shortly. Thank you for your patience! |
Everything looks good here. I am proceeding with the final acceptance. Congratulations! |
@editorialbot accept |
|
Ensure proper citation by uploading a plain text CITATION.cff file to the default branch of your repository. If using GitHub, a Cite this repository menu will appear in the About section, containing both APA and BibTeX formats. When exported to Zotero using a browser plugin, Zotero will automatically create an entry using the information contained in the .cff file. You can copy the contents for your CITATION.cff file here: CITATION.cff
If the repository is not hosted on GitHub, a .cff file can still be uploaded to set your preferred citation. Users will be able to manually copy and paste the citation. |
🐘🐘🐘 👉 Toot for this paper 👈 🐘🐘🐘 |
🚨🚨🚨 THIS IS NOT A DRILL, YOU HAVE JUST ACCEPTED A PAPER INTO JOSS! 🚨🚨🚨 Here's what you must now do:
Any issues? Notify your editorial technical team... |
🎉🎉🎉 Congratulations on your paper acceptance! 🎉🎉🎉 If you would like to include a link to your paper from your README use the following code snippets:
This is how it will look in your documentation: We need your help! The Journal of Open Source Software is a community-run journal and relies upon volunteer effort. If you'd like to support us please consider doing either one (or both) of the the following:
|
Submitting author: @VincentStimper (Vincent Stimper)
Repository: https://github.com/VincentStimper/normalizing-flows
Branch with paper.md (empty if default branch):
Version: v1.7.0
Editor: @mstimberg
Reviewers: @matejgrcic, @kazewong
Archive: 10.5281/zenodo.8027667
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
@matejgrcic & @kazewong, 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 @mstimberg 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 @kazewong
📝 Checklist for @matejgrcic
The text was updated successfully, but these errors were encountered: