Closed whedon closed 3 years ago
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @kinow, @xirdneh it looks like you're currently assigned to review this paper :tada:.
:warning: JOSS reduced service mode :warning:
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.
:star: Important :star:
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:
@whedon commands
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
@whedon generate pdf
PDF failed to compile for issue #2791 with the following error:
/app/vendor/ruby-2.4.4/lib/ruby/2.4.0/find.rb:43:in block in find': No such file or directory - 241d63aa7177374e42894fdd (Errno::ENOENT) from /app/vendor/ruby-2.4.4/lib/ruby/2.4.0/find.rb:43:in
collect!'
from /app/vendor/ruby-2.4.4/lib/ruby/2.4.0/find.rb:43:in find' from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-d14a699185fb/lib/whedon/processor.rb:66:in
find_paper_paths'
from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-d14a699185fb/bin/whedon:53:in prepare' from /app/vendor/bundle/ruby/2.4.0/gems/thor-0.20.3/lib/thor/command.rb:27:in
run'
from /app/vendor/bundle/ruby/2.4.0/gems/thor-0.20.3/lib/thor/invocation.rb:126:in invoke_command' from /app/vendor/bundle/ruby/2.4.0/gems/thor-0.20.3/lib/thor.rb:387:in
dispatch'
from /app/vendor/bundle/ruby/2.4.0/gems/thor-0.20.3/lib/thor/base.rb:466:in start' from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-d14a699185fb/bin/whedon:131:in
<top (required)>'
from /app/vendor/bundle/ruby/2.4.0/bin/whedon:23:in load' from /app/vendor/bundle/ruby/2.4.0/bin/whedon:23:in
@gkthiruvathukal hi, do you have karma to re-generate the article? I think the bot failed to do so, and I think I need it to complete the review checklist items for the paper. Please.
Thank you!
@whedon generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
Thanks!
@gkthiruvathukal Sorry I didn't get to your request as fast as @arfon, @kinow. Looks like we're good now.
:wave: @kinow, please update us on how your review is going.
:wave: @xirdneh, please update us on how your review is going.
My review is complete @gkthiruvathukal :heavy_check_mark:
Even managed to test s4rdm3x
with Apache Commons Imaging. I was going to use Apache Commons Lang, but I am currently working on Imaging. I didn't add all the relationships between packages, and the tool correctly highlighted the failed clusterings.
Didn't get fully familiar with the rest of the interface and metrics, but looks really useful for developers inspecting code bases, and getting familiar with new architectures.
I used IntelliJ's diagram builder to remind me of some of the relationships, but in the past I used Stan4J that would be a good tool to help users to create sysmdl
files.
Thank you for your patience and help with the issues found during the review @tobias-dv-lnu!
Awesome! @kinow: Do you consider the sysmdl for commons imaging to be "correct"? If so would can I include it in the repo as an available system (always on the lookout for more data :) )
And thank you for the feedback and found problems!
Awesome! @kinow: Do you consider the sysmdl for commons imaging to be "correct"? If so would can I include it in the repo as an available system (always on the lookout for more data :) )
Not yet, but I will open a draft pull request. We can discuss how to improve it. I didn't understand how to use all the options and settings, and for the experiment.xml
I simply used the same one from JebRef modifying the system loaded.
I've created a draft PR: https://github.com/tobias-dv-lnu/s4rdm3x/pull/14
Having some trouble testing a couple of things on the paper. I think it has to do with my local setup, I am currently tweaking some thing and will try to finish this over the weekend. Sorry for the delay.
@xirdneh: just let me know if I can help :)
@xirdneh Just checking on how things are coming along with your review. I'm definitely not trying to rush you but would like to see if we are getting closer.
@kinow Am I correct in interpreting your review as being complete and ready to recommend acceptance, or are their still issues requiring attention?
All good @gkthiruvathukal , review complete, article looking good, software built and tested locally. No other issues pending for me. Thanks!
@gkthiruvathukal I was able to test the code locally. Everything seems good. I am not able to edit the checklist. Should I just be able to edit the issue description?
@whedon re-invite @xirdneh as reviewer
The reviewer already has a pending invite.
@xirdneh please accept the invite by clicking this link: https://github.com/openjournals/joss-reviews/invitations
@xirdneh - please accept the repository invite above, this should then allow you to edit the checklist.
@xirdneh please let me know if there is anything I can do to help you with the review :)
Thanks for the nudge on this @tobias-dv-lnu.
@xirdneh Can you let us know your timeframe for completing your review? It looks like @kinow has done a fairly thorough review. I would like to move forward with acceptance as soon as we have your input. Thanks!
Hi, sorry about this.
I will finish the review by tomorrow afternoon.
I already have everything written down just needed the invitation.
@whedon re-invite @xirdneh as reviewer
I'm sorry @xirdneh, I'm afraid I can't do that. That's something only editors are allowed to do.
@gkthiruvathukal Sorry, could you send the invitation again? Seems like I didn't click it last time, I thought i did.
@whedon re-invite @xirdneh as reviewer
OK, the reviewer has been re-invited.
@xirdneh please accept the invite by clicking this link: https://github.com/openjournals/joss-reviews/invitations
@xirdneh I hope that works!
@xirdneh Thanks, it seems to have worked! :)
@gkthiruvathukal What are the next steps for this? According to the instructions as I understand them:
"Upon successful completion of the review, authors will make a tagged release of the software, and deposit a copy of the repository with a data-archiving service such as Zenodo or figshare, get a DOI for the archive, and update the review issue thread with the version number and DOI. "
or is there something else you want me to do?
@tobias-dv-lnu We are close! I will follow up with a final checklist as soon as I get a response to the following.
@xirdneh Can you please confirm for us that you have no further issues to raise regarding this submission? I would like to move toward acceptance and give final instructions.
@gkthiruvathukal Yes, the submission is complete.
Thanks, @xirdneh
@tobias-dv-lnu Ok, I am ready now! Please do the following:
Thanks!
@gkthiruvathukal Think I'm all done :)
Thanks @tobias-dv-lnu @gkthiruvathukal, it looks like this is ready for acceptance so I can take it from here.
@whedon generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@whedon set 10.5281/zenodo.4475664 as archive
OK. 10.5281/zenodo.4475664 is the archive.
@whedon set v1.3 as version
OK. v1.3 is the version.
@whedon accept
Attempting dry run of processing paper acceptance...
:wave: @openjournals/joss-eics, this paper is ready to be accepted and published.
Check final proof :point_right: https://github.com/openjournals/joss-papers/pull/2077
If the paper PDF and Crossref deposit XML look good in https://github.com/openjournals/joss-papers/pull/2077, then you can now move forward with accepting the submission by compiling again with the flag deposit=true
e.g.
@whedon accept deposit=true
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1145/3241403.3241456 is OK
MISSING DOIs
- 10.1016/j.infsof.2006.10.015 may be a valid DOI for title: Automated clustering to support the reflexion method
- 10.1109/wcre.2010.26 may be a valid DOI for title: Improving automated mapping in reflexion models using information retrieval techniques
- 10.1145/3344948.3344984 may be a valid DOI for title: Semi-automatic mapping of source code using naive Bayes
- 10.1145/3344948.3344997 may be a valid DOI for title: An exploration and experiment tool suite for code to architecture mapping techniques
- 10.1007/s11219-018-9404-z may be a valid DOI for title: Exploring the suitability of source code metrics for indicating architectural inconsistencies
- 10.1109/wcre.2012.35 may be a valid DOI for title: On the evolutionary nature of architectural violations
INVALID DOIs
- None
@tobias-dv-lnu, it looks like DOIs may be available but missing for a number of your references. Can you please check these, and add if appropriate? I will then be able to accept the submission. Thanks!
Thanks, @kyleniemeyer for helping to bring this to closure! :)
@kyleniemeyer Thanks! Just added the missing DOIs in the paper.bib file (whedon had made the correct guesses on the DOIs), everything should be ok now I hope :)
@whedon check references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.1016/j.infsof.2006.10.015 is OK
- 10.1109/WCRE.2010.26 is OK
- 10.1145/3344948.3344984 is OK
- 10.1145/3344948.3344997 is OK
- 10.1145/3241403.3241456 is OK
- 10.1007/s11219-018-9404-z is OK
- 10.1109/wcre.2012.35 is OK
MISSING DOIs
- None
INVALID DOIs
- None
Submitting author: @tobias-dv-lnu (Tobias Olsson) Repository: https://github.com/tobias-dv-lnu/s4rdm3x Version: v1.3 Editor: @gkthiruvathukal Reviewer: @kinow, @xirdneh Archive: 10.5281/zenodo.4475664
:warning: JOSS reduced service mode :warning:
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
@kinow & @xirdneh, 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 @gkthiruvathukal 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 @kinow
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @xirdneh
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper