-
-
Notifications
You must be signed in to change notification settings - Fork 38
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]: LGP: A robust Linear Genetic Programming implementation on the JVM using Kotlin. #1337
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @homonecloco it looks like you're currently assigned as the reviewer for this paper 🎉. ⭐ 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:
|
|
👋 @homonecloco - how are you getting along with your review? |
Hi, |
@homonecloco other than filling out the checkboxes that you think the submission satisfies, you can submit any other feedback/comments here in the issue thread. You can do that all at once, or as you have comments—either is fine! |
The documentation is extensive and detailed. However, the links to the API are broken. Reported as an issue in JedS6391/LGP#55 |
There is a link to the Thesis of the author, http://www.jedsimson.co.nz/static/assets/thesis/Open-Source%20Linear%20Genetic%20Programming.pdf Would it be appropriate to also include the supervisor as author? |
I had issues running the program, as stated JedS6391/LGP#56 . I'll continue the review once this issue is resolved. |
The installation consists on downloading the jars. There is also an option to do it from Maven. |
@JedS6391 I'm not sure whether you saw the issues that have been reported by @homonecloco. Can you please follow up when you get a chance? Thanks. |
Hi @gkthiruvathukal, JedS6391/LGP#55 and JedS6391/LGP#56 have been addressed. I am currently working on JedS6391/LGP#7, but it is taking a little while to write the unit tests before the issue can be closed. |
Thanks, @JedS6391. Please let us know when you've had a chance to add unit tests. |
Just a nudge here, @JedS6391. Any update on unit tests? |
@JedS6391 Please provide an update on when you will be able to address unit tests. |
Hi @gkthiruvathukal, apologies for the lack of updates. Unfortunately I have not had time recently to dedicate to the LGP project, and as such I am happy for this review to be cancelled and postponed until I can satisfy the requirements. |
@JedS6391 I don't want you to withdraw the submission. You have been good about addressing 2/3 issues. Can you give me an idea of when you might be able to do it? I will work with your schedule and nudge you accordingly. |
Much appreciated! I will able to start working on the remaining issues over the next few weekends. |
Hi @gkthiruvathukal -- just wanted to give you a brief progress update. I have implemented unit tests for a subset of the functionality in this PR. I hope to finish the remaining tests over the weekend. I will let you know once all unit tests have been completed. |
@JedS6391 Perfect. Let me know when you have a few more tests in place. I don't need 100% test coverage to proceed. I just did a quick look and it seems like something is failing, so I will wait until you get it sorted out. Thanks for moving on this! |
Hi @gkthiruvathukal, the latest commit against the develop branch has a green build and has a subset of the tests. At the moment, the line coverage is 20% and complexity coverage is 47%, but I intended to increase this going forward. If you are interested to see a breakdown of the areas covered, then that can be viewed here. I will continue to add more tests and eventually close the initial issue. |
@JedS6391 I find your efforts satisfactory as there is now a basis for doing further testing in future work. If our reviewer, @homonecloco, agrees, I would like to move toward accepting your submission. Once we hear from @homonecloco, you will need to create a Zenodo archive. |
@homonecloco When you get a chance, can you let us know if your issues are all addressed so I can proceed with the next steps (recommending acceptance). |
Hi, I haven't got a chance, I'd been travelling the last week. I'll aim to review before the end of this week. |
@homonecloco No problem! Thanks for your quick response. End of week is fine. |
@JedS6391 👋 ☝️ |
/ooo October 24 until October 27 |
Hi @Kevin-Mattheus-Moerman, I think something went wrong with the automatic upload to Zenodo from GitHub so I created the archive manually (see here). I have also updated the archive title + author list to be correct. The new DOI is 10.5281/zenodo.3519400. Apologies for the delay in getting this sorted. |
@whedon generate pdf |
|
I believe I need to change the DOI to the new one to get the paper to reflect the changes made? |
@JedS6391 no you need to make the changes in your paper.md file (https://github.com/JedS6391/LGP/blob/develop/paper.md) and run '@whedon generate pdf'. Note the two changes I proposed above. Let me know if you have questions. |
@whedon generate pdf |
|
Hi @Kevin-Mattheus-Moerman, the generated PDF now has the reflected changes (I didn't realise whedon read from the |
@whedon set 10.5281/zenodo.3519400 as archive |
OK. 10.5281/zenodo.3519400 is the archive. |
@whedon accept |
|
|
Check final proof 👉 openjournals/joss-papers#1057 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#1057, then you can now move forward with accepting the submission by compiling again with the flag
|
@whedon accept deposit=true |
|
🐦🐦🐦 👉 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 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! 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: @JedS6391 (Jed Simson)
Repository: https://github.com/JedS6391/LGP
Version: 5.1
Editor: @gkthiruvathukal
Reviewer: @homonecloco
Archive: 10.5281/zenodo.3519400
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) 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
@homonecloco, 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.theoj.org/about#reviewer_guidelines. Any questions/concerns please let @gkthiruvathukal know.
✨ Please try and complete your review in the next two weeks ✨
Review checklist for @homonecloco
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?The text was updated successfully, but these errors were encountered: