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]: tidyhydat: Extract and Tidy Canadian Hydrometric Data #509

Closed
whedon opened this issue Dec 15, 2017 · 16 comments
Closed

[PRE REVIEW]: tidyhydat: Extract and Tidy Canadian Hydrometric Data #509

whedon opened this issue Dec 15, 2017 · 16 comments
Assignees
Labels
pre-review R rOpenSci Submissions associated with rOpenSci TeX

Comments

@whedon
Copy link

whedon commented Dec 15, 2017

Submitting author: @boshek (Samuel Albers)
Repository: https://github.com/ropensci/tidyhydat
Version: 0.3.0
Editor: @arfon
Reviewer: @arfon

Author instructions

Thanks for submitting your paper to JOSS @boshek. 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.

@boshek 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 Dec 15, 2017

Hello human, I'm @whedon. I'm here to help you with some common editorial tasks.

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

@whedon commands

@whedon
Copy link
Author

whedon commented Dec 15, 2017

Attempting PDF compilation. Reticulating splines etc...

@whedon
Copy link
Author

whedon commented Dec 15, 2017

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

 /app/vendor/bundle/ruby/2.3.0/bundler/gems/whedon-453fa9fb981d/lib/whedon.rb:106:in `[]': no implicit conversion of String into Integer (TypeError)
	from /app/vendor/bundle/ruby/2.3.0/bundler/gems/whedon-453fa9fb981d/lib/whedon.rb:106:in `block in parse_affiliations'
	from /app/vendor/bundle/ruby/2.3.0/bundler/gems/whedon-453fa9fb981d/lib/whedon.rb:105:in `each'
	from /app/vendor/bundle/ruby/2.3.0/bundler/gems/whedon-453fa9fb981d/lib/whedon.rb:105:in `parse_affiliations'
	from /app/vendor/bundle/ruby/2.3.0/bundler/gems/whedon-453fa9fb981d/lib/whedon.rb:92:in `parse_authors'
	from /app/vendor/bundle/ruby/2.3.0/bundler/gems/whedon-453fa9fb981d/lib/whedon.rb:76:in `initialize'
	from /app/vendor/bundle/ruby/2.3.0/bundler/gems/whedon-453fa9fb981d/lib/whedon/processor.rb:26:in `new'
	from /app/vendor/bundle/ruby/2.3.0/bundler/gems/whedon-453fa9fb981d/lib/whedon/processor.rb:26:in `set_paper'
	from /app/vendor/bundle/ruby/2.3.0/bundler/gems/whedon-453fa9fb981d/bin/whedon:37:in `prepare'
	from /app/vendor/bundle/ruby/2.3.0/gems/thor-0.20.0/lib/thor/command.rb:27:in `run'
	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:in `dispatch'
	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-453fa9fb981d/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 `<main>'

@arfon
Copy link
Member

arfon commented Dec 15, 2017

@openjournals/ropensci - has this package already on-boarded? If so, could you share the thread please?

@arfon
Copy link
Member

arfon commented Dec 15, 2017

Scrub that. I found it myself :-) ropensci/software-review#152

@arfon
Copy link
Member

arfon commented Dec 15, 2017

@boshek - please merge this PR so we can proceed ropensci/tidyhydat#69

@boshek
Copy link

boshek commented Dec 15, 2017

@arfon Merged. Thanks!

@arfon
Copy link
Member

arfon commented Dec 15, 2017

@whedon generate pdf

@whedon
Copy link
Author

whedon commented Dec 15, 2017

Attempting PDF compilation. Reticulating splines etc...

@whedon
Copy link
Author

whedon commented Dec 15, 2017

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

   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100    16  100    16    0     0     39      0 --:--:-- --:--:-- --:--:--    39
Error producing PDF.
! LaTeX Error: File `fancyvrb.sty' not found.

Type X to quit or <RETURN> to proceed,
or enter new name. (Default extension: sty)

Enter file name: 
! Emergency stop.
<read *> 
         
l.153 \newcommand

Looks like we failed to compile the PDF

@arfon
Copy link
Member

arfon commented Dec 15, 2017

@whedon assign @arfon as editor

@whedon
Copy link
Author

whedon commented Dec 15, 2017

OK, the editor is @arfon

@arfon
Copy link
Member

arfon commented Dec 15, 2017

@whedon assign @arfon as reviewer

@whedon whedon assigned arfon and unassigned arfon Dec 15, 2017
@whedon
Copy link
Author

whedon commented Dec 15, 2017

OK, the reviewer is @arfon

@arfon
Copy link
Member

arfon commented Dec 15, 2017

@whedon start review magic-word=bananas

@arfon arfon closed this as completed Dec 15, 2017
@whedon
Copy link
Author

whedon commented Dec 15, 2017

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

@arfon arfon added the rOpenSci Submissions associated with rOpenSci label Feb 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pre-review R rOpenSci Submissions associated with rOpenSci TeX
Projects
None yet
Development

No branches or pull requests

3 participants