-
-
Notifications
You must be signed in to change notification settings - Fork 39
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]: surtvep: An R package for estimating time-varying effects #5688
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:
|
Checking the BibTeX entries failed with the following error:
|
|
Wordcount for |
Failed to discover a valid open source license |
Review checklist for @turgeonmaximeConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Review checklist for @adibenderConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@osorensen I work in the field and have a package which provides similar functionality. Not sure if this is conflict of interest, but I don't think it will affect review. Just wanted to mention it here |
@adibender, I agree. I see this as a strength rather than a conflict of interest, but thanks for mentioning it. |
@LingfengLuo0510 I just read the manuscript for the first time, and I have a few questions/comments.
I'm sure I'll have further comments/questions as I keep reviewing the package, but I'll put them in a different comment! |
Also, I haven't been able to install the package using the instructions in the manuscript. Here is the error message I got. r$> remotes::install_github('UM-KevinHe/surtvep')
Downloading GitHub repo UM-KevinHe/surtvep@HEAD
── R CMD build ──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
checking for file ‘/private/var/folders/hb/r5l2r5b12gg5y2zxyg4l7nh80000gn/T/Rtmp42cwos/remotes1030ea884ab0/UM-KevinHe-surtvep-31f9fd4/DESC✔ checking for file ‘/private/var/folders/hb/r5l2r5b12gg5y2zxyg4l7nh80000gn/T/Rtmp42cwos/remotes1030ea884ab0/UM-KevinHe-surtvep-31f9fd4/DESCRIPTION’
─ preparing ‘surtvep’:
✔ checking DESCRIPTION meta-information ...
─ cleaning src
─ checking for LF line-endings in source and make files and shell scripts
─ checking for empty or unneeded directories
─ building ‘surtvep_1.0.0.tar.gz’
* installing *source* package ‘surtvep’ ...
** using staged installation
** libs
using C++ compiler: ‘Apple clang version 14.0.3 (clang-1403.0.22.14.1)’
using C++11
using SDK: ‘MacOSX13.3.sdk’
clang++ -arch arm64 -std=gnu++11 -I"/Library/Frameworks/R.framework/Resources/include" -DNDEBUG -I'/Library/Frameworks/R.framework/Versions/4.3-arm64/Resources/library/Rcpp/include' -I'/Library/Frameworks/R.framework/Versions/4.3-arm64/Resources/library/RcppArmadillo/include' -I/opt/R/arm64/include -fPIC -falign-functions=64 -Wall -g -O2 -c PenalizeStopCpp.cpp -o PenalizeStopCpp.o
PenalizeStopCpp.cpp:4:10: fatal error: 'omp.h' file not found
#include <omp.h>
^~~~~~~
1 error generated.
make: *** [PenalizeStopCpp.o] Error 1
ERROR: compilation failed for package ‘surtvep’
* removing ‘/Library/Frameworks/R.framework/Versions/4.3-arm64/Resources/library/surtvep’
* restoring previous ‘/Library/Frameworks/R.framework/Versions/4.3-arm64/Resources/library/surtvep’
Warning message:
In i.p(...) :
installation of package ‘/var/folders/hb/r5l2r5b12gg5y2zxyg4l7nh80000gn/T//Rtmp42cwos/file1030e79df6e36/surtvep_1.0.0.tar.gz’ had non-zero exit status I did notice that the instructions in the For reference, here's my session information: r$> sessionInfo()
R version 4.3.0 (2023-04-21)
Platform: aarch64-apple-darwin20 (64-bit)
Running under: macOS Ventura 13.4.1
Matrix products: default
BLAS: /Library/Frameworks/R.framework/Versions/4.3-arm64/Resources/lib/libRblas.0.dylib
LAPACK: /Library/Frameworks/R.framework/Versions/4.3-arm64/Resources/lib/libRlapack.dylib; LAPACK version 3.11.0
locale:
[1] en_CA.UTF-8/en_CA.UTF-8/en_CA.UTF-8/C/en_CA.UTF-8/en_CA.UTF-8
time zone: America/Edmonton
tzcode source: internal
attached base packages:
[1] stats graphics grDevices utils datasets methods base
loaded via a namespace (and not attached):
[1] processx_3.8.2 compiler_4.3.0 R6_2.5.1 rprojroot_2.0.3 cli_3.6.1 prettyunits_1.1.1 tools_4.3.0
[8] curl_5.0.1 crayon_1.5.2 remotes_2.4.2 desc_1.4.2 callr_3.7.3 ps_1.7.5 pkgbuild_1.4.2 |
@LingfengLuo0510 One more thing: I ran |
Thanks a lot for you comments @turgeonmaxime! |
@LingfengLuo0510, I just wanted to add to the comments from @turgeonmaxime that although we don't require R package submissions to be on CRAN, they should in general pass |
@LingfengLuo0510, you're welcome to start addressing the points raised by @turgeonmaxime. Please keep us updated about any progress. |
@osorensen Thanks a lot for the instructions. I am submitting this project to CRAN now. I will update when it is successfully uploaded. |
@LingfengLuo0510 some things that are currently missing from your package:
|
Thanks @adibender. @LingfengLuo0510, any updates on fixing the issues pointed out be @turgeonmaxime or on the CRAN submission? |
Thank you for your comments. The package is currently under CRAN review. I’ve addressed CRAN's feedback and resubmitted the updated package for approval. I will provide an update once the review process is complete. |
👋 @LingfengLuo0510 - thanks for all of your hard work on this. Your paper is very clean and that is much appreciated! I just need you to address a few final things before we accept this for publication:
Let me know when these are done and I will move to accept. |
Thanks a lot! Thanks again! |
@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 new publication @LingfengLuo0510! Many thanks to @osorensen for editing and @adibender and @turgeonmaxime for your time, hard work, and expertise!! JOSS wouldn't be able to function nor succeed without your efforts. Please consider becoming a reviewer for JOSS if you are not already: https://reviewers.joss.theoj.org/join |
🎉🎉🎉 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:
|
@crvernon I feel so sorry that may I ask if I can update the middle name of one of the authors? So sorry about this. |
Yes please do so. I am getting ready to board a flight but will reaccept this as soon as I can. Just let me know once you have made the change. |
@editorialbot generate pdf |
@editorialbot generate pdf |
@editorialbot generate pdf |
@crvernon Than However, I'm encountering errors when generating the PDF, even after reverting to the previously accepted version. Could this be related to the paper's accepted status? |
@editorialbot generate pdf |
@editorialbot reaccept |
|
🌈 Paper updated! New PDF and metadata files 👉 openjournals/joss-papers#5551 |
@LingfengLuo0510 - the paper should now reflect your changes. |
🎉🎉🎉 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: @LingfengLuo0510 (Lingfeng Luo)
Repository: https://github.com/UM-KevinHe/surtvep
Branch with paper.md (empty if default branch): JOSS
Version: v1.0.0
Editor: @osorensen
Reviewers: @adibender, @turgeonmaxime
Archive: 10.5281/zenodo.12575049
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
@adibender & @turgeonmaxime, 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 @osorensen 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 @turgeonmaxime
📝 Checklist for @adibender
The text was updated successfully, but these errors were encountered: