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]: PyPVRPM: Photovoltaic Reliability and Performance Model in Python #4093
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @SarthakJariwala it looks like you're currently assigned to review this paper 🎉. Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post. ⭐ Important ⭐ 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:
|
Wordcount for |
|
|
OK, @blthayer is now a reviewer |
Hi @blthayer and @SarthakJariwala. Thank you for being a reviewer for JOSS. At the top of this thread you will see a checklist for you to use as a guideline for your review. At JOSS we generally work with the authors to help them get their submission into publishable form (if it is not already), so you can interact with them here or on their repo regarding any changes you think are needed. Remember that the review is publish and non-anonymous, so is meant to be a collaborative effort, and your github handles will appear on the final PDF as reviewers, so you'll get credit for your efforts. Let's aim to have this done in a about 1 month? So Feb 28th? |
FYI (@brandons209, @jgostick): At this point I plan to start my review in earnest either this weekend or next week. Looking forward to checking out |
@blthayer Great! If you need any help or want some example cases for testing let me know. |
👋 @SarthakJariwala, please update us on how your review is going (this is an automated reminder). |
@jgostick - it looks like I need a fresh invitation sent in order to be able to edit my checklist. When I went to the following URL, it said the invitation was expired: https://github.com/openjournals/joss-reviews/invitations This isn't (yet) blocking me from performing my review. Thanks! |
My review is currently blocked by FSEC-Photovoltaics/pvrpm-lcoe#36, @brandons209 |
|
|
@brandons209 Congrats. An editor in chief will be notified that the submission is ready to go, and will make it official shortly. |
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#3055 If the paper PDF and the deposit XML files look good in openjournals/joss-papers#3055, then you can now move forward with accepting the submission by compiling again with the command |
@jgostick Is there a reason one checkbox is unchecked? @brandons209 Please edit the metadata in your Zenodo archive so that the title and author list match your JOSS paper. Also it looks like the version listed here and in the Zenodo archive are not the same, so please rectify that and let me know what version you want here. |
@kthyng Alright, all the information should be updated on Zenodo. Version 1.7.4 is correct. |
@brandons209 Your software title is "PyPVRPM: Photovoltaic Reliability and Performance Model in Python" right? I'm looking for that to be the Zenodo archive title here, unless you prefer not to have that: https://zenodo.org/record/6355115 |
@kthyng That's correct. I didn't realize the archive had to match the paper exactly. I'll change it. |
@brandons209 - I'll proofread the paper now (as the AEiC on duty this week). |
👋 @jgostick & @SarthakJariwala - I see one checkbox (functionality) unchecked. Can you discuss why this is the case? |
@brandons209 - I'm suggesting some small language changes in FSEC-Photovoltaics/pvrpm-lcoe#61 - please merge this or let me know what you disagree with. Once we do this, and get the last checkbox issue resolved, we can move to final acceptance and publication |
HI @danielskatz |
@jgostick - thanks - I missed that earlier discussion. |
@danielskatz Looks good, I merged the PR. |
@editorialbot recommend-accept |
|
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#3079 If the paper PDF and the deposit XML files look good in openjournals/joss-papers#3079, then you can now move forward with accepting the submission by compiling again with the command |
@editorialbot accept |
|
🐦🐦🐦 👉 Tweet 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 to @brandons209 and co-authors!! And thanks to @blthayer and @SarthakJariwala for reviewing, and @jgostick for editing! |
🎉🎉🎉 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: @brandons209 (Brandon Silva)
Repository: https://github.com/FSEC-Photovoltaics/pvrpm-lcoe
Branch with paper.md (empty if default branch):
Version: 1.7.4
Editor: @jgostick
Reviewers: @blthayer, @SarthakJariwala
Archive: 10.5281/zenodo.6355115
Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.
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
@SarthakJariwala, 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 @jgostick 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 ✨
Review checklist for @SarthakJariwala
✨ Important: Please do not use the Convert to issue functionality when working through this checklist, instead, please open any new issues associated with your review in the software repository associated with the submission. ✨
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @blthayer
✨ Important: Please do not use the Convert to issue functionality when working through this checklist, instead, please open any new issues associated with your review in the software repository associated with the submission. ✨
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
The text was updated successfully, but these errors were encountered: