openjournals / joss-reviews

Reviews for the Journal of Open Source Software
Creative Commons Zero v1.0 Universal
721 stars 38 forks source link

[REVIEW]: The MCAPL Framework including the Agent Infrastructure Layer and Agent JavaPathfinder #617

Closed whedon closed 6 years ago

whedon commented 6 years ago

Submitting author: @louiseadennis (Louise Dennis) Repository: https://sourceforge.net/p/mcapl/mcapl_code/ci/master/tree/ Version: 2017 Editor: @danielskatz Reviewer: @BasTesterink, @fraimondi Archive: 10.5281/zenodo.1237635

Status

status

Status badge code:

HTML: <a href="http://joss.theoj.org/papers/9f735950f5f68052fbda99e41f215cb0"><img src="http://joss.theoj.org/papers/9f735950f5f68052fbda99e41f215cb0/status.svg"></a>
Markdown: [![status](http://joss.theoj.org/papers/9f735950f5f68052fbda99e41f215cb0/status.svg)](http://joss.theoj.org/papers/9f735950f5f68052fbda99e41f215cb0)

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 1 instructions & questions

@BasTesterink, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:

  1. Make sure you're logged in to your GitHub account
  2. Be sure to accept the invite at this URL: https://github.com/openjournals/joss-reviews/invitations

The reviewer guidelines are available here: https://joss.theoj.org/about#reviewer_guidelines. Any questions/concerns please let @danielskatz know.

Conflict of interest

Code of Conduct

General checks

Functionality

Documentation

Software paper

Reviewer 2 instructions & questions

@fraimondi, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:

  1. Make sure you're logged in to your GitHub account
  2. Be sure to accept the invite at this URL: https://github.com/openjournals/joss-reviews/invitations

The reviewer guidelines are available here: https://joss.theoj.org/about#reviewer_guidelines. Any questions/concerns please let @danielskatz know.

Conflict of interest

Code of Conduct

General checks

Functionality

Documentation

Software paper

whedon commented 6 years ago

Hello human, I'm @whedon. I'm here to help you with some common editorial tasks. @BasTesterink it looks like you're currently assigned as the reviewer for this paper :tada:.

: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:

  1. Set yourself as 'Not watching' https://github.com/openjournals/joss-reviews:

watching

  1. You may also like to change your default settings for this watching repositories in your GitHub profile here: https://github.com/settings/notifications

notifications

For a list of things I can do to help you, just type:

@whedon commands
whedon commented 6 years ago
Attempting PDF compilation. Reticulating splines etc...
whedon commented 6 years ago

PDF failed to compile for issue #617 with the following error:

/app/vendor/ruby-2.3.4/lib/ruby/2.3.0/find.rb:43:in block in find': No such file or directory (Errno::ENOENT) from /app/vendor/ruby-2.3.4/lib/ruby/2.3.0/find.rb:43:incollect!' from /app/vendor/ruby-2.3.4/lib/ruby/2.3.0/find.rb:43:in find' from /app/vendor/bundle/ruby/2.3.0/bundler/gems/whedon-a31a5c2a9125/lib/whedon/processor.rb:52:infind_paper_paths' from /app/vendor/bundle/ruby/2.3.0/bundler/gems/whedon-a31a5c2a9125/bin/whedon:32:in prepare' from /app/vendor/bundle/ruby/2.3.0/gems/thor-0.20.0/lib/thor/command.rb:27:inrun' from /app/vendor/bundle/ruby/2.3.0/gems/thor-0.20.0/lib/thor/invocation.rb:126:in invoke_command' from /app/vendor/bundle/ruby/2.3.0/gems/thor-0.20.0/lib/thor.rb:387:indispatch' from /app/vendor/bundle/ruby/2.3.0/gems/thor-0.20.0/lib/thor/base.rb:466:in start' from /app/vendor/bundle/ruby/2.3.0/bundler/gems/whedon-a31a5c2a9125/bin/whedon:99:in<top (required)>' from /app/vendor/bundle/ruby/2.3.0/bin/whedon:22:in load' from /app/vendor/bundle/ruby/2.3.0/bin/whedon:22:in

'

danielskatz commented 6 years ago

@arfon - can you compile this paper again, as you did in https://github.com/openjournals/joss-reviews/issues/573

Can you also make sure both reviewers (Reviewer1: @BasTesterink; Reviewer2: @fraimondi) are invited and able to access the checklist?

danielskatz commented 6 years ago

I'm also just noting here that @fraimondi may need until April 20 to perform this review

arfon commented 6 years ago

@louiseadennis - I'm afraid I'm unable to download the code from the repository. The git clone operation keeps timing out. Also, browsing the files, I don't seem to be able to find the paper.md and paper.bib files. Could you please link to them here?

arfon commented 6 years ago

Can you also make sure both reviewers (Reviewer1: @BasTesterink; Reviewer2: @fraimondi) are invited and able to access the checklist?

Done.

louiseadennis commented 6 years ago

@arfon the paper.md and paper.bib files are in doc/joss_paper https://sourceforge.net/p/mcapl/mcapl_code/ci/master/tree/doc/joss_paper/paper.bib and https://sourceforge.net/p/mcapl/mcapl_code/ci/master/tree/doc/joss_paper/paper.md

I've no idea what is going on with sourceforge. I can't clone either, though I can fetch and push.

louiseadennis commented 6 years ago

@arfon as far as I can work out, cloning is failing because of a memory problem at the sourceforge end. I can't ssh into sourceforge at the moment to try the fixes suggested online (source forge says it has disable shell while it transitions to a new datacenter) and while there are some largish files in the repo (jars of libs etc.) I don't think I have loads of huge files that could be causing this. I'm a little stumped how to proceed here but will look into it further on Thursday when I have more time.

arfon commented 6 years ago

Thanks @louiseadennis - I've managed to compile the paper using the direct links you provided here: 10.21105.joss.00617.pdf

BasTesterink commented 6 years ago

I have very little to comment on this submission. Within minutes I was able to download and compile the framework on my Windows 10 machine with an Eclipse Oxygen installation. The examples that I tried (there are a lot of them) all ran without any errors on the first try. The code is well documented, clean and readable, both for the core Java code and the example agent-programming code. The manual contains extensive explanations of the kind of software this is and discusses the examples on the different application domains (those being the different agent programming languages that the framework supports).

I'm not sure what the expectations from JOSS are on community guidelines. There is no explicit guidelines sections in the readme for instance. I checked the review box because for me it seemed clear that any questions/suggestions should go to @loiseadennis or though the SF ticket system or mailing list.

In conclusion, I believe the contribution to be a valuable piece of software for agent-oriented programmers. The MCAPL project is one of the few up-to-date initiatives on agent program verification which gets more and more relevant now that AI & Ethics is attracting attention.

danielskatz commented 6 years ago

Thanks @BasTesterink

I'm not sure what the expectations from JOSS are on community guidelines. There is no explicit guidelines sections in the readme for instance. I checked the review box because for me it seemed clear that any questions/suggestions should go to @loiseadennis or though the SF ticket system or mailing list.

There should be an explicit guidelines section somewhere, likely in the README. It's useful for people new to open source to have some text that tells them how they can ask questions, or how they can make contributions (both in terms of mechanism and policy).

louiseadennis commented 6 years ago

Not 100% sure of the process here. But I've added a "Problems and Contributions" section to the README and pushed to the master branch.

danielskatz commented 6 years ago

For some reason, https://sourceforge.net/p/mcapl/mcapl_code/ci/master/tree/ isn't currently working

danielskatz commented 6 years ago

now it's back - not sure what happened...

louiseadennis commented 6 years ago

It was behaving strangely when @arfon was looking at it on Wednesday, throwing errors which implied a memory management problem at some level. I was going to look into it but then the problem went away so I assumed sourceforge had fixed it at their end.

danielskatz commented 6 years ago

Just adding a note that @fraimondi's review is expected the 20th of April - we'll be on hold until then.

fraimondi commented 6 years ago

I have done my review and I am happy with the repository, the documentation and the paper. The installation and examples all worked first time. In addition to Eclipse, I tried from the command line in Linux and Mac: no problem .

Very minor point (may be useful for bots?): the LICENSE file is called COPYING, is this OK?

danielskatz commented 6 years ago

The license file should be called LICENSE for consistency with general practices and user/developer expectations. @louiseadennis - can you fix this? Then we can move forward with acceptance.

danielskatz commented 6 years ago

and thanks @fraimondi !

danielskatz commented 6 years ago

👋 @louiseadennis - note this is waiting for a relatively small change from you.

louiseadennis commented 6 years ago

It's on the stack, we're just in the middle of project marking here. Should get to it this week or early next week at the very latest.

louiseadennis commented 6 years ago

Changed and pushed to the repository!

danielskatz commented 6 years ago

@fraimondi - I thought this was ready to be accepted, but my final look shows that there is one box left unchecked - yours on functionality.

Can you tell us what you think is needed?

fraimondi commented 6 years ago

Apologies, my mystake! The only thing that is needed is a new pair of glasses for the reviewer. I've now checked all the boxes, I think it is ready to be accepted. Thanks!

danielskatz commented 6 years ago

👋 @arfon - can we move this to acceptance now?

arfon commented 6 years ago

@louiseadennis - At this point could you make an archive of the reviewed software in Zenodo/figshare/other service and update this thread with the DOI of the archive? I can then move forward with accepting the submission.

louiseadennis commented 6 years ago

I've added a new version to the existing Zenodo archive at https://doi.org/10.5281/zenodo.1237635

Is that sufficient or do I need to generate a new archive altogether?

arfon commented 6 years ago

Is that sufficient or do I need to generate a new archive altogether?

This should be good.

arfon commented 6 years ago

@whedon set 10.5281/zenodo.1237635 as archive

whedon commented 6 years ago

OK. 10.5281/zenodo.1237635 is the archive.

arfon commented 6 years ago

@BasTesterink, @fraimondi - many thanks for your reviews here and to @danielskatz for editing this submission ✨

@louiseadennis - your paper is now accepted into JOSS and your DOI is https://doi.org/10.21105/joss.00617 ⚡️:rocket: :boom:

whedon commented 6 years ago

:tada::tada::tada: Congratulations on your paper acceptance! :tada::tada::tada:

If you would like to include a link to your paper from your README use the following code snippet:

[![DOI](http://joss.theoj.org/papers/10.21105/joss.00617/status.svg)](https://doi.org/10.21105/joss.00617)

This is how it will look in your documentation:

DOI

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:

louiseadennis commented 6 years ago

I just noticed a typo in the title of the paper (an instead of and). I've fixed in the repository. Is there anyway this can be fixed online?

arfon commented 6 years ago

I just noticed a typo in the title of the paper (an instead of and). I've fixed in the repository. Is there anyway this can be fixed online?

I've just recompiled the paper so this should be fixed now.

louiseadennis commented 6 years ago

Thanks!