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]: ThermoCodegen: a python/C++ package for the generation of custom thermodynamic models #4874
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 |
|
👋 @mspieg @bocklund @rdguha1995 the actual review will take place here. Per the instructions above, when you are ready to do your review, comment with |
Review checklist for @rdguha1995Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Hi @bocklund and @rdguha1995, just wanted to check on the status of your review. Would it be possible to wrap these up before the end of the month / year? Thanks! |
Thank you. I was beginning to wonder…
Marc
…Sent from my iPhone
On Dec 12, 2022, at 8:09 AM, Kyle Niemeyer ***@***.***> wrote:
Hi @bocklund and @rdguha1995, just wanted to check on the status of your review. Would it be possible to wrap these up before the end of the month / year? Thanks!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.
|
Hey Team, Apologies for the delay. I have gone through most of the paper, but I am left with some of the functional checks. I should be able to knock them off before the break. |
Hey @mspieg , I am having some issues with the installation and I have opened an issue at the repository. Let me know what is going wrong here and I can continue the review from there. |
sure…thanks…
I’ll try to get to this today (but if not, both Cian and I will be in Chicago for AGU and will figure out what’s going on).
More soon
Marc
… On Dec 12, 2022, at 12:41 PM, rdguha1995 ***@***.***> wrote:
Hey @mspieg <https://github.com/mspieg> , I am having some issues with the installation and I have opened an issue at the repository <https://gitlab.com/ENKI-portal/ThermoCodegen/-/issues/12>. Let me know what is going wrong here and I can continue the review from there.
—
Reply to this email directly, view it on GitHub <#4874 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABKNDSGLH3S7V36BPPWBVRTWM5PVLANCNFSM6AAAAAARMOJ2QE>.
You are receiving this because you were mentioned.
-------------------------------------------------
Marc Spiegelman
Dept of Earth and Env Sciences
Dept of App. Physics and App. Math
Columbia University
-------------------------------------------------
|
Hey Team, I think I have gone over all the checklist items in the review. As mentioned in the issue I have opened in the target repository, the only sticking point I have is the convoluted interdependencies users might face if they decide to install the package locally. The docker containers are working great, which is what the advisable usage case is (as mentioned in the documentation), but it would be great if the authors can provide streamlined instructions for local installation as well in their documentation. A couple of potential suggestions from my end are:-
Besides the complications in the installation, everything else looks good to me. Great job @mspieg and team for the comprehensive documentation and examples. |
Thanks @rdguha1995 for the comments! @mspieg have you been able to take a look at the above issues? |
Hi @bocklund, do you anticipate being able to complete your review soon? |
Hi @kyleniemeyer, |
This comment was marked as outdated.
This comment was marked as outdated.
@editorialbot set v1.0.0 as version |
Done! version is now v1.0.0 |
Hi @mspieg, there is a minor issue with the Connolly 2009 reference ("n/a-n/a" as the pages). Looking at the PDF at https://agupubs.onlinelibrary.wiley.com/doi/epdf/10.1029/2009GC002540 it seems they would like "Q10014" listed as the page number. Could you fix that? |
Sure…thanks for catching that.
… On Jun 30, 2023, at 1:26 PM, Kyle Niemeyer ***@***.***> wrote:
Hi @mspieg <https://github.com/mspieg>, there are some minor issues with the Connolly 2009 reference ("n/a-n/a" as the pages). Looking at the PDF at https://agupubs.onlinelibrary.wiley.com/doi/epdf/10.1029/2009GC002540 it seems they would like "Q10014" listed as the page number. Could you fix that?
—
Reply to this email directly, view it on GitHub <#4874 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABKNDSGI2X6BP5KNUIFAXXTXN4D4FANCNFSM6AAAAAARMOJ2QE>.
You are receiving this because you were mentioned.
-------------------------------------------------
Marc Spiegelman
Dept of Earth and Env Sciences
Dept of App. Physics and App. Math
Columbia University
-------------------------------------------------
|
done
… On Jun 30, 2023, at 1:26 PM, Kyle Niemeyer ***@***.***> wrote:
Hi @mspieg <https://github.com/mspieg>, there are some minor issues with the Connolly 2009 reference ("n/a-n/a" as the pages). Looking at the PDF at https://agupubs.onlinelibrary.wiley.com/doi/epdf/10.1029/2009GC002540 it seems they would like "Q10014" listed as the page number. Could you fix that?
—
Reply to this email directly, view it on GitHub <#4874 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABKNDSGI2X6BP5KNUIFAXXTXN4D4FANCNFSM6AAAAAARMOJ2QE>.
You are receiving this because you were mentioned.
-------------------------------------------------
Marc Spiegelman
Dept of Earth and Env Sciences
Dept of App. Physics and App. Math
Columbia University
-------------------------------------------------
|
@editorialbot generate pdf |
@editorialbot recommend-accept |
|
|
👋 @openjournals/pe-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#4365, then you can now move forward with accepting the submission by compiling again with the command |
@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 @mspieg on your article's publication in JOSS! If you haven't already, please consider signing up as a future reviewer. On my end, I'm seeing a website error for this paper, so I'm going to leave this issue open until that resolves. Many thanks to @bocklund and @rdguha1995 for reviewing this submission. |
On Jun 30, 2023, at 3:17 PM, Kyle Niemeyer ***@***.***> wrote:
Congratulations @mspieg <https://github.com/mspieg> on your article's publication in JOSS! If you haven't already, please consider signing up as a future reviewer <https://reviewers.joss.theoj.org/join>.
On my end, I'm seeing a website error for this paper, so I'm going to leave this issue open until that resolves.
Thanks…I was just going to send you a note about the web-site error. I’ll keep a tab on things until they resolve.
Thank you for all your help with this paper
cheers
marc
… Many thanks to @bocklund <https://github.com/bocklund> and @rdguha1995 <https://github.com/rdguha1995> for reviewing this submission.
—
Reply to this email directly, view it on GitHub <#4874 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABKNDSAYYCUUIUXNQ2GTZCTXN4Q3NANCNFSM6AAAAAARMOJ2QE>.
You are receiving this because you were mentioned.
-------------------------------------------------
Marc Spiegelman
Dept of Earth and Env Sciences
Dept of App. Physics and App. Math
Columbia University
-------------------------------------------------
|
@editorialbot reaccept |
|
🌈 Paper updated! New PDF and metadata files 👉 openjournals/joss-papers#4368 |
@mspieg all looks good now! |
🎉🎉🎉 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:
|
Yes…thank you for fixing that…and finally done!
Cheers
Marc
… On Jul 1, 2023, at 3:20 PM, Kyle Niemeyer ***@***.***> wrote:
@mspieg <https://github.com/mspieg> all looks good now!
—
Reply to this email directly, view it on GitHub <#4874 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABKNDSD64YMBAMTY5UERRWTXOBZ6RANCNFSM6AAAAAARMOJ2QE>.
You are receiving this because you were mentioned.
-------------------------------------------------
Marc Spiegelman
Dept of Earth and Env Sciences
Dept of App. Physics and App. Math
Columbia University
-------------------------------------------------
|
Submitting author: @mspieg (Marc Spiegelman)
Repository: https://gitlab.com/ENKI-portal/ThermoCodegen
Branch with paper.md (empty if default branch): joss_paper
Version: v1.0.0
Editor: @kyleniemeyer
Reviewers: @bocklund, @rdguha1995
Archive: 10.5281/zenodo.8102054
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
@bocklund & @rdguha1995, 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 @kyleniemeyer 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 @rdguha1995
📝 Checklist for @bocklund
The text was updated successfully, but these errors were encountered: