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]: PyQMRI: An accelerated Python based Quantitative MRI Python toolbox #2718

Closed
whedon opened this issue Oct 2, 2020 · 39 comments

Comments

@whedon
Copy link

whedon commented Oct 2, 2020

Submitting author: @MaierOli2010 (Oliver Maier)
Repository: https://github.com/IMTtugraz/PyQMRI
Version: v0.3.1
Editor: @Kevin-Mattheus-Moerman
Reviewers: @grlee77, @agahkarakuzu, @DARSakthi
Managing EiC: Kevin M. Moerman

⚠️ JOSS reduced service mode ⚠️

Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.

Author instructions

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

@MaierOli2010 if you have any suggestions for potential reviewers then please mention them here in this thread (without tagging them with an @). In addition, this list of people have already agreed to review for JOSS and may be suitable for this submission (please start at the bottom of the list).

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 Oct 2, 2020

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

⚠️ JOSS reduced service mode ⚠️

Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.

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

@whedon
Copy link
Author

whedon commented Oct 2, 2020

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

/app/vendor/ruby-2.4.4/lib/ruby/2.4.0/find.rb:43:in block in find': No such file or directory - tmp/2718 (Errno::ENOENT) from /app/vendor/ruby-2.4.4/lib/ruby/2.4.0/find.rb:43:in collect!'
from /app/vendor/ruby-2.4.4/lib/ruby/2.4.0/find.rb:43:in find' from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-364ded062842/lib/whedon/processor.rb:61:in find_paper_paths'
from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-364ded062842/bin/whedon:50: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-364ded062842/bin/whedon:119: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

'

@Kevin-Mattheus-Moerman
Copy link
Member

@whedon generate pdf

@whedon
Copy link
Author

whedon commented Oct 2, 2020

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

/app/vendor/ruby-2.4.4/lib/ruby/2.4.0/find.rb:43:in block in find': No such file or directory - tmp/2718 (Errno::ENOENT) from /app/vendor/ruby-2.4.4/lib/ruby/2.4.0/find.rb:43:in collect!'
from /app/vendor/ruby-2.4.4/lib/ruby/2.4.0/find.rb:43:in find' from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-364ded062842/lib/whedon/processor.rb:61:in find_paper_paths'
from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-364ded062842/bin/whedon:50: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-364ded062842/bin/whedon:119: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

'

@Kevin-Mattheus-Moerman
Copy link
Member

@whedon generate pdf from branch JOSS_pub

@whedon
Copy link
Author

whedon commented Oct 2, 2020

Attempting PDF compilation from custom branch JOSS_pub. Reticulating splines etc...

@whedon
Copy link
Author

whedon commented Oct 2, 2020

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

sh: 1: cd: can't cd to tmp/2718
/app/vendor/ruby-2.4.4/lib/ruby/2.4.0/find.rb:43:in block in find': No such file or directory - tmp/2718 (Errno::ENOENT) from /app/vendor/ruby-2.4.4/lib/ruby/2.4.0/find.rb:43:in collect!'
from /app/vendor/ruby-2.4.4/lib/ruby/2.4.0/find.rb:43:in find' from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-364ded062842/lib/whedon/processor.rb:61:in find_paper_paths'
from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-364ded062842/bin/whedon:50: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-364ded062842/bin/whedon:119: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

'

@Kevin-Mattheus-Moerman
Copy link
Member

@openjournals/dev can you help troubleshoot why this paper is not compiling? Thanks

@Kevin-Mattheus-Moerman
Copy link
Member

@whedon check repository

@Kevin-Mattheus-Moerman
Copy link
Member

@whedon generate pdf from branch JOSS_pub

@whedon
Copy link
Author

whedon commented Oct 2, 2020

Attempting PDF compilation from custom branch JOSS_pub. Reticulating splines etc...

@whedon
Copy link
Author

whedon commented Oct 2, 2020

👉📄 Download article proof 📄 View article proof on GitHub 📄 👈

@Kevin-Mattheus-Moerman
Copy link
Member

@openjournals/dev can you help troubleshoot why this paper is not compiling? Thanks

I think I fixed it by changing the repository link to be the repo itself not the branch

@Kevin-Mattheus-Moerman
Copy link
Member

@whedon check repository

@whedon
Copy link
Author

whedon commented Oct 2, 2020

Software report (experimental):

github.com/AlDanial/cloc v 1.84  T=0.55 s (98.9 files/s, 38614.8 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Python                          30           1662           3655           8461
C                                8            679            356           5650
reStructuredText                 9            134             98            231
DOS Batch                        1              8              1             27
Dockerfile                       1              7              1             27
YAML                             2              8             11             26
make                             1              4              6             10
INI                              1              0              0             10
Bourne Shell                     1              0              0              8
-------------------------------------------------------------------------------
SUM:                            54           2502           4128          14450
-------------------------------------------------------------------------------


Statistical information for the repository '2718' was gathered on 2020/10/02.
The following historical commit information, by author, was found:

Author                     Commits    Insertions      Deletions    % of changes
Maier, Oliver                  341        257592         241085           89.13
Oliver                          62         30982           6909            6.77
Oliver Maier                   131         16345           6600            4.10

Below are the number of rows from each author that have survived and are still
intact in the current revision:

Author                     Rows      Stability          Age       % in comments
Maier, Oliver             20463            7.9          7.4                4.81

@MaierOli2010
Copy link

I would suggest hanayik as potential reviewer given his knowledge in MRI and Image Analysis and sbacchio for his knowledge in HPC and Iterative Linear Solvers. vitorsr, ulvgard, and stsievert might be suitable too given their background in optimization and image processing.

@Kevin-Mattheus-Moerman
Copy link
Member

@whedon invite @jni as editor

@whedon
Copy link
Author

whedon commented Oct 3, 2020

@jni has been invited to edit this submission.

@jni
Copy link

jni commented Oct 3, 2020

@Kevin-Mattheus-Moerman unfortunately I don't have the bandwidth right now, especially since I'm about to go on leave. I will however additionally recommend grlee77 as an additional potential reviewer.

@Kevin-Mattheus-Moerman
Copy link
Member

@whedon assign @Kevin-Mattheus-Moerman as editor

@whedon
Copy link
Author

whedon commented Oct 3, 2020

OK, the editor is @Kevin-Mattheus-Moerman

@Kevin-Mattheus-Moerman
Copy link
Member

@JoshKarpel
Copy link

Unfortunately I don't have time at the moment. Thanks for asking!

@spinicist
Copy link

spinicist commented Oct 4, 2020

Looks very interesting but I don't have access to a suitable GPU!

@grlee77
Copy link

grlee77 commented Oct 4, 2020

Sure, I am interested.

@Kevin-Mattheus-Moerman
Copy link
Member

@whedon assign @grlee77 as reviewer

@whedon
Copy link
Author

whedon commented Oct 5, 2020

OK, @grlee77 is now a reviewer

@62442katieb
Copy link

@Kevin-Mattheus-Moerman looks like an interesting toolbox, but I don't have access to a GPU at the moment

@agahkarakuzu
Copy link

@Kevin-Mattheus-Moerman I've arranged GPU access, happy to help out!

@mathieuboudreau
Copy link

Since @agahkarakuzu and I are from the same lab, I yield to him and decline for myself.

@DARSakthi
Copy link

DARSakthi commented Oct 5, 2020 via email

@Kevin-Mattheus-Moerman
Copy link
Member

@whedon add @agahkarakuzu as reviewer

@whedon
Copy link
Author

whedon commented Oct 5, 2020

OK, @agahkarakuzu is now a reviewer

@Kevin-Mattheus-Moerman
Copy link
Member

@DARSakthi if you are still interested I'd like to add you as a third reviewer, let me know. Thanks!

@DARSakthi
Copy link

DARSakthi commented Oct 6, 2020 via email

@Kevin-Mattheus-Moerman
Copy link
Member

@whedon add @DARSakthi as reviewer

@whedon
Copy link
Author

whedon commented Oct 6, 2020

OK, @DARSakthi is now a reviewer

@Kevin-Mattheus-Moerman
Copy link
Member

@whedon start review

@whedon
Copy link
Author

whedon commented Oct 7, 2020

OK, I've started the review over in #2727.

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