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]: MatD3: A Database and Online Presentation Package for Research Data Supporting Materials Discovery, Design, and Dissemination #1919

Closed
whedon opened this issue Nov 28, 2019 · 56 comments

Comments

@whedon
Copy link

whedon commented Nov 28, 2019

Submitting author: @raullaasner (Raul Laasner)
Repository: https://github.com/HybriD3-database/MatD3
Version: 1.0.0
Editor: @majensen
Reviewers: @dgasmith, @mkhorton

Author instructions

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

@raullaasner 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 28, 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 28, 2019

Attempting to check references...

@whedon
Copy link
Author

whedon commented Nov 28, 2019

Attempting PDF compilation. Reticulating splines etc...

@whedon
Copy link
Author

whedon commented Nov 28, 2019

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

/app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-df8b50fe58b8/lib/whedon/author.rb:68:in block in build_affiliation_string': Problem with affiliations for Xiaochen Du, perhaps the affiliations index need quoting? (RuntimeError) from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-df8b50fe58b8/lib/whedon/author.rb:67:in each'
from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-df8b50fe58b8/lib/whedon/author.rb:67:in build_affiliation_string' from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-df8b50fe58b8/lib/whedon/author.rb:17:in initialize'
from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-df8b50fe58b8/lib/whedon.rb:201:in new' from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-df8b50fe58b8/lib/whedon.rb:201:in block in parse_authors'
from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-df8b50fe58b8/lib/whedon.rb:198:in each' from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-df8b50fe58b8/lib/whedon.rb:198:in parse_authors'
from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-df8b50fe58b8/lib/whedon.rb:91: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 28, 2019


OK DOIs

- None

MISSING DOIs

- https://doi.org/10.1038/sdata.2019.2 may be missing for title: Qresp, a tool for curating, discovering and
 exploring reproducible scientific papers
- https://doi.org/10.1063/1.4812323 may be missing for title: Commentary: The Materials Project: A materials
 genome approach to accelerating materials
 innovation
- https://doi.org/10.1007/s11837-016-2001-3 may be missing for title: The Materials Data Facility: Data services to
 advance materials science research
- https://doi.org/10.1016/j.commatsci.2012.02.005 may be missing for title: AFLOW: An automatic framework for high-throughput
 materials discovery
- https://doi.org/10.1007/s11837-013-0755-4 may be missing for title: Materials Design and Discovery with High-Throughput
 Density Functional Theory: The Open Quantum
 Materials Database (OQMD)

INVALID DOIs

- None

@Kevin-Mattheus-Moerman
Copy link
Member

@openjournals/dev can you see what is wrong here? The affiliation index does use quotes.

@Kevin-Mattheus-Moerman
Copy link
Member

@raullaasner while trouble shooting this you can try compiling your paper here too: https://whedon.theoj.org/
If it is compiling you can run @whedon generate pdf here

@xuanxu
Copy link
Member

xuanxu commented Nov 30, 2019

@Kevin-Mattheus-Moerman, @raullaasner: There was an error with the indexing of the affiliations, this PR should fix the compiling error.

@Kevin-Mattheus-Moerman
Copy link
Member

Thanks @xuanxu !

@Kevin-Mattheus-Moerman
Copy link
Member

@raullaasner can you check the above fix by @xuanxu ?

@raullaasner
Copy link

@Kevin-Mattheus-Moerman Yes, sorry for the delay. This is fixed now.

@Kevin-Mattheus-Moerman
Copy link
Member

@whedon generate pdf

@whedon
Copy link
Author

whedon commented Nov 30, 2019

Attempting PDF compilation. Reticulating splines etc...

@whedon
Copy link
Author

whedon commented Nov 30, 2019

@Kevin-Mattheus-Moerman
Copy link
Member

@raullaasner can you also work on those missing DOI's? You can run @whedon check references and @whedon generate pdf here too to check things.

@Kevin-Mattheus-Moerman
Copy link
Member

@trallard @csoneson @majensen @akeshavan could one of you edit this submission? Thanks!

@majensen
Copy link
Member

majensen commented Dec 1, 2019

@whedon assign @majensen as editor

@whedon
Copy link
Author

whedon commented Dec 1, 2019

OK, the editor is @majensen

@raullaasner
Copy link

raullaasner commented Dec 1, 2019

@Kevin-Mattheus-Moerman I added DOIs to the paper references except for the SpringerMaterials and Nomad citations, which are only web links. It seems there is no better way to cite these at this point.

@raullaasner
Copy link

Regarding the proof, would it possible have "3" as a superscript in MatD3? In the paper, I had written MatD<sup>3</sup> everywhere but this formatting seems not to be present in the proof.

@Kevin-Mattheus-Moerman
Copy link
Member

@whedon generate pdf

@whedon
Copy link
Author

whedon commented Dec 1, 2019

Attempting PDF compilation. Reticulating splines etc...

@whedon
Copy link
Author

whedon commented Dec 1, 2019

@Kevin-Mattheus-Moerman
Copy link
Member

@openjournals/dev do you know how to use superscript in the text as the author requests?

Regarding the proof, would it possible have "3" as a superscript in MatD3? In the paper, I had written MatD<sup>3</sup> everywhere but this formatting seems not to be present in the proof.

@pythonpanda2
Copy link

pythonpanda2 commented Dec 10, 2019

@majensen

I was reviewing the JOSS Conflict of Interest (COI) Policy. I regret to inform you that my current employment at ANL and an ongoing collaboration with submitter's research group might lead to COI.

I would like to nominate @aykol for review.

@majensen
Copy link
Member

Thanks for your diligence on that issue @pythonpanda2 -- @aykol are you willing to review this submission?

@pythonpanda2
Copy link

pythonpanda2 commented Dec 10, 2019

@majensen I would also nominate @dgasmith

@majensen
Copy link
Member

Hi @dgasmith - would you be willing to review this submission to JOSS?

@dgasmith
Copy link

@majensen Yes, happy to review the submission.

@majensen
Copy link
Member

@whedon assign @dgasmith as reviewer

@whedon whedon assigned dgasmith and majensen and unassigned majensen and pythonpanda2 Dec 10, 2019
@whedon
Copy link
Author

whedon commented Dec 10, 2019

OK, the reviewer is @dgasmith

@computron
Copy link

Hey @majensen , sorry but I am unable to perform this review

@dgasmith
Copy link

@computron /wave!

I could also suggest @mkhorton.

@mkhorton
Copy link

I'd be happy to take a look at this, I can't make a strong commitment on time however.

@majensen
Copy link
Member

Thanks for your response @computron -
@mkhorton - thx - what would be a workable timeline for you?

@mkhorton
Copy link

mkhorton commented Dec 11, 2019 via email

@majensen
Copy link
Member

@mkhorton understood. I will start the review, but @raullaasner I expect we all will be off and on until 1 Jan - does this sound ok?

@majensen
Copy link
Member

@whedon add @mkhorton as reviewer

@whedon whedon assigned dgasmith, majensen and mkhorton and unassigned majensen and dgasmith Dec 11, 2019
@whedon
Copy link
Author

whedon commented Dec 11, 2019

OK, @mkhorton is now a reviewer

@majensen
Copy link
Member

Applying the tacit approval principle, I will start this review. Thanks all.

@majensen
Copy link
Member

@whedon start review

@whedon
Copy link
Author

whedon commented Dec 12, 2019

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

@aykol
Copy link

aykol commented Dec 13, 2019

Sorry for the delayed response. I'm glad new reviewers were found, as I wasn't going to be able to review it.

@majensen
Copy link
Member

Thanks for following up @aykol !

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