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]: Cocktail Shaker: An open source drug expansion and enumeration library for peptides #1916

Closed
whedon opened this issue Nov 25, 2019 · 49 comments

Comments

@whedon
Copy link

whedon commented Nov 25, 2019

Submitting author: @Sulstice (Suliman Sharif)
Repository: https://github.com/Sulstice/cocktail-shaker
Version: v1.1.0
Editor: @csoneson
Reviewers: @floux, @olivertomic

Author instructions

Thanks for submitting your paper to JOSS @Sulstice. Currently, there isn't an JOSS editor assigned to your paper.

@Sulstice 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 Nov 25, 2019

Hello human, I'm @whedon, a robot that can help you with some common editorial tasks.

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

What happens now?

This submission is currently in a pre-review state which means we are waiting for an editor to be assigned and for them to find some reviewers for your submission. This may take anything between a few hours to a couple of weeks. Thanks for your patience 😸

You can help the editor by looking at this list of potential reviewers to identify individuals who might be able to review your submission (please start at the bottom of the list). Also, feel free to suggest individuals who are not on this list by mentioning their GitHub handles here.

@whedon
Copy link
Author

whedon commented Nov 25, 2019

Attempting to check references...

@whedon
Copy link
Author

whedon commented Nov 25, 2019

Attempting PDF compilation. Reticulating splines etc...

@whedon
Copy link
Author

whedon commented Nov 25, 2019

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

/app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-df8b50fe58b8/lib/whedon.rb:145:in check_fields': Paper YAML header is missing expected fields: affiliations (RuntimeError) from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-df8b50fe58b8/lib/whedon.rb:87:in initialize'
from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-df8b50fe58b8/lib/whedon/processor.rb:36:in new' from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-df8b50fe58b8/lib/whedon/processor.rb:36:in set_paper'
from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-df8b50fe58b8/bin/whedon:55: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-df8b50fe58b8/bin/whedon:116: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

'

@whedon
Copy link
Author

whedon commented Nov 25, 2019


OK DOIs

- None

MISSING DOIs

- None

INVALID DOIs

- None

@Kevin-Mattheus-Moerman
Copy link
Member

@Sulstice your paper seems to be missing some fields (e.g. affiliations).

Have a look here:
https://joss.readthedocs.io/en/latest/submitting.html#checking-that-your-paper-compiles

You can test compile your paper here:
https://whedon.theoj.org/

@Sulstice
Copy link

Hi @Kevin-Mattheus-Moerman I just updated the affiliations to "No affiliations" since I currently have none. Does that suffice or is there another protocol I should follow?

@arfon
Copy link
Member

arfon commented Nov 26, 2019

@Sulstice - you can add an affiliation of 'None' but this need to be in the affiliations block as described in the example paper: https://joss.readthedocs.io/en/latest/submitting.html#example-paper-and-bibliography

@Kevin-Mattheus-Moerman
Copy link
Member

Kevin-Mattheus-Moerman commented Nov 28, 2019

@csoneson @will-rowe @majensen could one of you edit this submission? thanks

@majensen
Copy link
Member

@whedon generate pdf

@whedon
Copy link
Author

whedon commented Nov 29, 2019

Attempting PDF compilation. Reticulating splines etc...

@whedon
Copy link
Author

whedon commented Nov 29, 2019

@csoneson
Copy link
Member

csoneson commented Dec 2, 2019

@whedon assign @csoneson as editor

@whedon
Copy link
Author

whedon commented Dec 2, 2019

OK, the editor is @csoneson

@csoneson
Copy link
Member

csoneson commented Dec 2, 2019

Hi @Sulstice - could you check Figure 1 in your paper? It doesn't seem to show up.

@arfon
Copy link
Member

arfon commented Dec 2, 2019

@Sulstice - you'll need to link to the image URL like this rather than the relative path you're currently using:

https://raw.githubusercontent.com/Sulstice/cocktail-shaker/master/images/figure_1.png

@Sulstice
Copy link

Sulstice commented Dec 3, 2019

@csoneson @arfon Made the change!

@arfon
Copy link
Member

arfon commented Dec 3, 2019

@whedon generate pdf

@whedon
Copy link
Author

whedon commented Dec 3, 2019

Attempting PDF compilation. Reticulating splines etc...

@whedon
Copy link
Author

whedon commented Dec 3, 2019

@arfon
Copy link
Member

arfon commented Dec 3, 2019

@Sulstice - Could you move the references you currently have in the paper.md file into a paper.bib file and cite them directly please? (You can read how to do that here)

@Sulstice
Copy link

Sulstice commented Dec 4, 2019

@arfon Done!

@arfon
Copy link
Member

arfon commented Dec 4, 2019

@whedon generate pdf

@whedon
Copy link
Author

whedon commented Dec 4, 2019

Attempting PDF compilation. Reticulating splines etc...

@whedon
Copy link
Author

whedon commented Dec 4, 2019

@csoneson
Copy link
Member

csoneson commented Dec 4, 2019

Thanks @Sulstice. I will take care of your submission, and start by finding suitable reviewers. If you have suggestions (e.g., from the list linked in the first post above), please let me know.

@Sulstice
Copy link

Sulstice commented Dec 5, 2019

@csoneson I looked through the list

Username: mskwark, road2stat, semacu seem most reasonable since they are in the peptide/cheminformatics space.

@Sulstice
Copy link

Sulstice commented Dec 5, 2019

Should I be doing a release of the software so y'all can pip install it? Or should I wait pending on whether you approve or not?

@csoneson
Copy link
Member

csoneson commented Dec 6, 2019

@Sulstice - yes, I think that would be nice.

@csoneson
Copy link
Member

csoneson commented Dec 6, 2019

@whedon add @floux as reviewer

@whedon whedon assigned csoneson and unassigned csoneson Dec 6, 2019
@whedon
Copy link
Author

whedon commented Dec 6, 2019

OK, @floux is now a reviewer

@csoneson
Copy link
Member

csoneson commented Dec 6, 2019

@mskwark, @semacu - would one of you be willing to review this submission for JOSS?

@Sulstice
Copy link

Sulstice commented Dec 7, 2019

@csoneson I released the cocktail shaker v1.1.0 and it is labeled as a "pre-release" for now pending the JOSS approval.

You should be able to easily install with this command pip install cocktail-shaker==1.1.0

@csoneson
Copy link
Member

Ping @mskwark, @semacu

@mskwark, @semacu - would one of you be willing to review this submission for JOSS?

@csoneson
Copy link
Member

csoneson commented Dec 14, 2019

@olivertomic, @nanxstats - would one of you be willing to review this submission for JOSS?

@olivertomic
Copy link

@csoneson , I will be unavailable until mid-february. Would that be too late?

@nanxstats
Copy link

Thanks @csoneson for the invitation but I'm a bit busy with a personal project right now and in the next few months. My apologies. Maybe next time...

@csoneson
Copy link
Member

Thanks @olivertomic - I'm hoping to start the review soon, so I will reach out to a couple of more potential reviewers, but I may come back to you :)

@csoneson
Copy link
Member

@diegoenry, @matthiaslein - would one of you be willing to review this submission for JOSS?

@csoneson
Copy link
Member

@fdroessler, @keipertk - would one of you be interested in reviewing this submission for JOSS?

@csoneson
Copy link
Member

👋 @Sulstice - I suspect we may have more luck finding available reviewers after the holidays and the often hectic end of semester time. If you would be ok with a slight delay, and @olivertomic is still available from mid-February, that is one option. Otherwise I suggest we continue looking for reviewers in early January, after the holidays.

@floux - thanks for your patience! I'll start the actual review issue as soon as we have one more reviewer.

@Sulstice
Copy link

Sulstice commented Dec 28, 2019

@csoneson No worries! I'm not in a rush and can wait! Please let me know if there is anything I can do to help as well.

@csoneson
Copy link
Member

csoneson commented Jan 3, 2020

👋 @olivertomic - Happy New Year! Would you still be interested and available in mid-February to review this submission for JOSS?

Cocktail Shaker: An open source drug expansion and enumeration library for peptides

@olivertomic
Copy link

@csoneson - Happy New Year to you too! Sure, I would be still interested to do it from the proposed time.

@csoneson
Copy link
Member

csoneson commented Jan 4, 2020

@olivertomic - brilliant, thanks! Then I'll add you as a reviewer and start the review issue. I'll also ask our bot whedon to send an automated reminder in mid-February.

@csoneson
Copy link
Member

csoneson commented Jan 4, 2020

@whedon add @olivertomic as reviewer

@whedon whedon assigned csoneson and olivertomic and unassigned csoneson Jan 4, 2020
@whedon
Copy link
Author

whedon commented Jan 4, 2020

OK, @olivertomic is now a reviewer

@csoneson
Copy link
Member

csoneson commented Jan 4, 2020

@whedon start review

@whedon
Copy link
Author

whedon commented Jan 4, 2020

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

@csoneson csoneson closed this as completed Jan 4, 2020
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

8 participants