Skip to content
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

[PRE REVIEW]: Nashpy: A Python library for the computation of Nash equilibria #761

Closed
whedon opened this issue Jun 4, 2018 · 38 comments
Closed

Comments

@whedon
Copy link

whedon commented Jun 4, 2018

Submitting author: @drvinceknight (Vincent Knight)
Repository: https://github.com/drvinceknight/Nashpy
Version: v0.0.14
Editor: @labarba
Reviewers: @Fil, @alex-konovalov

Author instructions

Thanks for submitting your paper to JOSS @drvinceknight. The JOSS editor (shown at the top of this issue) will work with you on this issue to find a reviewer for your submission before creating the main review issue.

@drvinceknight if you have any suggestions for potential reviewers then please mention them here in this thread. In addition, this list of people have already agreed to review for JOSS and may be suitable for this submission.

Editor instructions

The JOSS submission bot @whedon is here to help you find and assign reviewers and start the main review. To find out what @whedon can do for you type:

@whedon commands
@whedon
Copy link
Author

whedon commented Jun 4, 2018

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

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

@whedon commands

@whedon
Copy link
Author

whedon commented Jun 4, 2018

Attempting PDF compilation. Reticulating splines etc...

@whedon
Copy link
Author

whedon commented Jun 4, 2018

--> Check article proof 📄 <--

@arfon
Copy link
Member

arfon commented Jun 4, 2018

👋 @labarba - the submitting author suggested you as the handling editor.

@labarba
Copy link
Member

labarba commented Jun 17, 2018

@drvinceknight In several places (repo, docs), you have a note like "to do sophisticated equilibria computation you should use gambit" — could you explain how this software, which you say computes equilibria in 2-player games only, would be useful in a research scenario?

@labarba
Copy link
Member

labarba commented Jun 17, 2018

After inspecting the software repository and the documentation, I get the clear impression that this package has more of a pedagogical value than a research application. You recently reviewed one of our first submissions to JOSE (thanks!) ... that software originally came to JOSS—have a look at the pre-review discussion: #212
Do you think your submission here is in a similar situation? (That would be my impression.)

@drvinceknight
Copy link

After inspecting the software repository and the documentation, I get the clear impression that this package has more of a pedagogical value than a research application. You recently reviewed one of our first submissions to JOSE (thanks!) ... that software originally came to JOSS—have a look at the pre-review discussion: #212
Do you think your submission here is in a similar situation? (That would be my impression.)

I actually was hesitating in my choice for this one: I'll happily resubmit this over to JOSE :)

@labarba
Copy link
Member

labarba commented Jun 18, 2018

👍 I do think you should include a couple of worked-out examples, in that case. For JOSE, one thing we want is to make it easier for someone else to adopt the software/materials, or for someone to use them for self-learning. Worked examples are important for that.

@drvinceknight
Copy link

I do think you should include a couple of worked-out examples, in that case. For JOSE, one thing we want is to make it easier for someone else to adopt the software/materials, or for someone to use them for self-learning. Worked examples are important for that.

Yup of course, I'm at a conference at the moment but will reshape the paper for JOSE. In terms of worked examples, do you mean in paper.md? The library documentation has quite an extensive set of examples.

A thought: I've actually used Nashpy for a game theory course which uses software in a lot of other ways as well that I was thinking as writing up as a paper for JOSE also. (Use of gh-pages as content delivery, heavy reliability on jupyter notebooks with a testing framework, bespoke static site generator, as well as sphinx backed documentation of "how to lead the course".) My intent is that the paper on Nashpy and the course would not be the same.

@drvinceknight
Copy link

I have submitted this to JOSE so am closing this. Thanks 👍

@drvinceknight
Copy link

drvinceknight commented Jul 15, 2018

Dear editors,

I would like to un-"withdraw" this paper, please let me know if it's simpler if I were to resubmit it perhaps?

This follows a discussion at JOSE openjournals/jose-reviews#18 were I believe it has become clear that a paper about this software library is not within the scope of JOSE.

I apologise for the confusion/mistake.

@labarba regarding your queries prior to withdrawal:

@drvinceknight In several places (repo, docs), you have a note like "to do sophisticated equilibria computation you should use gambit" — could you explain how this software, which you say computes equilibria in 2-player games only, would be useful in a research scenario?

I have modified the paper to hopefully make this clearer, the following is now included:

# Statement of need

Access to these algorithms is non trival, an example of this includes the
modelling of healthcare decisions [@knight2017measuring] where a bespoke
theoretic result was used to design a specific algorithm for the computation of
equilibria. Easily accessibly software would make that research more
straightforward as no new algorithm would need to be implemented.

The most mature piece of software available for the computation of equilibria
is **Gambit** [@mckelvey2006gambit]. Gambit includes a python wrapper to its
core C functionality however is not currently portable for example
Windows is not supported. There does exist a web interface with a Gambit back
end: [Game theory
explorer](http://gte.csc.liv.ac.uk/index/index.html#document-documentation)
however this is not practical for reproducible research.

``Nashpy`` is a Python library with all dependencies being part of the standard
scientific Python stack (numpy and scipy [@scipy]) thus it is portable. For
example Windows support is regularly tested through a Windows continuous
integration service (Appveyor).

I don't believe that there being another piece of software that has the same functionality precludes from publication in JOSS but I might be mistaken. EDIT: I note for example http://joss.theoj.org/papers/0cc2d656d8a02a84e0cde905207820d7 which has the same functionality as https://github.com/marcharper/python-ternary and http://joss.theoj.org/papers/225837fff5f6f153660cd05044cd4267 which has the same functionality as http://ciw.readthedocs.io/ and http://simpy.readthedocs.io/en/latest/ (I've picked out two papers from my own areas of expertise there :)).

Apologies again for the confusion and extra work that has resulted.

@drvinceknight drvinceknight reopened this Jul 15, 2018
@arfon arfon removed the withdrawn label Jul 23, 2018
@arfon
Copy link
Member

arfon commented Jul 23, 2018

I would like to un-"withdraw" this paper, please let me know if it's simpler if I were to resubmit it perhaps?

OK, thanks for the heads up @drvinceknight. We should be able to work with this original submission.

@arfon
Copy link
Member

arfon commented Jul 23, 2018

@whedon generate pdf

@whedon
Copy link
Author

whedon commented Jul 23, 2018

Attempting PDF compilation. Reticulating splines etc...

@whedon
Copy link
Author

whedon commented Jul 23, 2018

@arfon
Copy link
Member

arfon commented Jul 23, 2018

@drvinceknight - looks like some of the LaTeX formatting is a little wonky in the compiled PDF ☝️. Could you try and fix that? If you make a change to your paper.md you can ask @whedon to recompile the paper with @whedon generate pdf

@drvinceknight
Copy link

Sorry for the late reply @arfon, I've had a week's leave. Thanks for taking a look at this :) Regenerating now 👍

@drvinceknight
Copy link

@whedon generate pdf

@whedon
Copy link
Author

whedon commented Jul 30, 2018

Attempting PDF compilation. Reticulating splines etc...

@whedon
Copy link
Author

whedon commented Jul 30, 2018

@drvinceknight
Copy link

That's looking good to me, let me know if there's anything else I should/could do at this stage.

@drvinceknight
Copy link

@arfon @labarba I realise everyone is busy so no problem at all (and apologies for the pings), just checking if I there's anything I can do and/or should be doing to start the ball rolling on the review process? (In case perhaps this has been forgotten).

@arfon
Copy link
Member

arfon commented Aug 10, 2018

Hi @drvinceknight - things are pretty busy at JOSS right now (close to 100 submissions on the go!) so we're a little slow getting to things.

Can you look at this list of potential reviewers and suggest a few people that might be suitable to review your package?

@drvinceknight
Copy link

No problem at all, I completely understand: zero pressure/impatience on my side.

Looking through the list I'd suggest https://github.com/fil looks like a good fit, otherwise https://github.com/jordigh or https://github.com/nicoguaro perhaps.

Thank you again to all the editors for your efforts/work on this journal.

@Fil
Copy link

Fil commented Aug 18, 2018

@labarba I'm in

@labarba
Copy link
Member

labarba commented Aug 18, 2018

@whedon assign @Fil as reviewer

@whedon
Copy link
Author

whedon commented Aug 18, 2018

OK, the reviewer is @Fil

@labarba
Copy link
Member

labarba commented Aug 18, 2018

Thanks, @Fil — I'm waiting for a reply from a possible second reviewer, then I'll start the review!

@labarba
Copy link
Member

labarba commented Aug 18, 2018

@whedon add @alex-konovalov as reviewer

@whedon whedon assigned olexandr-konovalov, Fil and labarba and unassigned Fil and labarba Aug 18, 2018
@whedon
Copy link
Author

whedon commented Aug 18, 2018

OK, @alex-konovalov is now a reviewer

@labarba
Copy link
Member

labarba commented Aug 18, 2018

@whedon start review

@labarba
Copy link
Member

labarba commented Aug 18, 2018

Hmm ... @arfon, our loyal bot @whedon is not obeying my command to start the review. What's going on?

@arfon
Copy link
Member

arfon commented Aug 19, 2018

@whedon commands

@whedon
Copy link
Author

whedon commented Aug 19, 2018

Here are some things you can ask me to do:

# List all of Whedon's capabilities
@whedon commands

# Assign a GitHub user as the sole reviewer of this submission
@whedon assign @username as reviewer

# Add a GitHub user to the reviewers of this submission
@whedon add @username as reviewer

# Remove a GitHub user from the reviewers of this submission
@whedon remove @username as reviewer

# List of editor GitHub usernames
@whedon list editors

# List of reviewers together with programming language preferences and domain expertise
@whedon list reviewers

# Change editorial assignment
@whedon assign @username as editor

# Set the software archive DOI at the top of the issue e.g.
@whedon set 10.0000/zenodo.00000 as archive

# Open the review issue
@whedon start review

🚧 🚧 🚧 Experimental Whedon features 🚧 🚧 🚧

# Compile the paper
@whedon generate pdf

@arfon
Copy link
Member

arfon commented Aug 19, 2018

@whedon start review

1 similar comment
@arfon
Copy link
Member

arfon commented Aug 19, 2018

@whedon start review

@whedon
Copy link
Author

whedon commented Aug 19, 2018

OK, I've started the review over in #904. Feel free to close this issue now!

@arfon
Copy link
Member

arfon commented Aug 19, 2018

OK, I've started the review over in #904. Feel free to close this issue now!

@labarba - I found a weird bug with this paper. I've fixed it now.

@arfon arfon closed this as completed Aug 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants