Joss-reviews: [PRE REVIEW]: infotheory: A C++/Python package for multivariate information theoretic analysis

Created on 7 Jul 2019  ยท  46Comments  ยท  Source: openjournals/joss-reviews

Submitting author: @madvn (Madhavun Candadai)
Repository: https://github.com/madvn/infotheory
Version: 1.0
Editor: @poulson
Reviewers: ajgates42, @Autoplectic

Author instructions

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

@madvn 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
C++ Python TeX pre-review

Most helpful comment

All 46 comments

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

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 :smile_cat:

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.

Attempting PDF compilation. Reticulating splines etc...

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

ORCID looks to be the wrong length
/app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-85c572552d68/lib/whedon.rb:123:in block in check_orcids': Problem with ORCID (https://orcid.org/0000-0002-9254-8641) for Madhavun Candadai (RuntimeError) from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-85c572552d68/lib/whedon.rb:121:ineach'
from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-85c572552d68/lib/whedon.rb:121:in check_orcids' from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-85c572552d68/lib/whedon.rb:81:ininitialize'
from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-85c572552d68/lib/whedon/processor.rb:36:in new' from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-85c572552d68/lib/whedon/processor.rb:36:inset_paper'
from /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-85c572552d68/bin/whedon:55:in prepare' from /app/vendor/bundle/ruby/2.4.0/gems/thor-0.20.3/lib/thor/command.rb:27:inrun'
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:indispatch'
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-85c572552d68/bin/whedon:116:in 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

'

๐Ÿ‘‹ @ madvn โ€” your paper is not compiling; the error appears to be with an ORCID. Can you try to fix, then run @whedon generate pdf here to try the compilation again?

๐Ÿ‘‹ Thanks, @labarba I have updated ORCIDs

@whedon generate pdf

Attempting PDF compilation. Reticulating splines etc...

@usethedata โ€” would you be able to handle this submission as editor?

@madvn โ€” the paper at the moment feels like a "wall of text." Could you make an extra effort to separate the text into sub-sections, and build up to the specialized content a little more gradually? Bear in mind our documentation says that you should "describing the high-level functionality and purpose of the software for a diverse, non-specialist audience."

@labarba I'm a bit strapped at the moment, with ongoing elder care issues and a major proposal due at the end of July. If there's anyone else that can do it, that might be better. Otherwise, I can do it, but I'll be slower in getting to things than I'd prefer.

@poulson โ€” Might you be able to handle this submission? A bit out of your landscape, but no one editor is perfect for this one and you have scads of C++ experience.

Thanks, @labarba. I have reorganized the paper into sections and included more high-level description of the package.

@whedon generate pdf

Attempting PDF compilation. Reticulating splines etc...

Hi @labarba, I think I could handle this submission.

@whedon assign @poulson as editor

OK, the editor is @poulson

:wave: Alexander (@ajgates42): Given your expertise in python and information theory, you seem like an ideal reviewer for this submission. Would you have the time and interest to help review this submission? And, would you happen to have any suggestions for other reviewers?

:wave: Fabian (@fsteeg): You have listed expertise in information retrieval, though your languages were given as Java and Scala. Would you feel confident as a reviewer of a C++/Python package on information theory? If so, would you have the time to review?

:wave: Andre (@andremrsantos): You listed experience in python/C++ and in statistics and bioinformatics. Would you feel confident reviewing a submission on information theory? If so, do you have the time and interest in doing so?

Hi @poulson,
although I would love to contribute. My knowledge of information theory is limited and I wouldn't feel confident on review submission on the subject.
I hope you find your reviewers soon.

Thanks for the invite @poulson. Looks like a fun project, I would be happy to review it.

@ajgates42 Thanks! Looking forward to hearing your feedback on the project.

@poulson We are continuing to use this project in our research and we have discovered that with some minor changes the multivariate analyses methods in this package can be extended from bi-variate to tri-variate analyses. This would significantly increase the value of the package because, to our knowledge, there are no published packages for tri-variate analyses. Since we are in the pre-review stage, would it be okay for me to update it real quick (in a day or two)?

@whedon add ajgates42 as reviewer

OK, ajgates42 is now a reviewer

Hi @madvn, the purpose of the review is to improve the quality of the software and documentation, so I think waiting for the update would be okay, given that the review hasn't yet started.

I will reach out to a few more potential reviewers in parallel with those changes if that makes sense to you.

:wave: Fabian (@fsteeg) -- You have listed expertise in information retrieval, though your languages were given as Java and Scala. Would you feel confident as a reviewer of a C++/Python package on information theory? If so, would you have the time to review?

@ajgates42 Do you happen to have any further reviewers you would suggest? Our list of information theory experts is modest and it seems you are from this community. Any help here would greatly aid the review process -- and therefore the work itself.

I think Ryan James ([email protected], @Autoplectic) would be a great
person to have assess the contributions of this package.

Another good fit would be Georg Martius ([email protected]
, @georgmartius)

On Mon, Jul 22, 2019 at 11:18 AM Jack Poulson notifications@github.com
wrote:

๐Ÿ‘‹ Fabian (@fsteeg https://github.com/fsteeg) -- You have listed
expertise in information retrieval, though your languages were given as
Java and Scala. Would you feel confident as a reviewer of a C++/Python
package on information theory? If so, would you have the time to review?

@ajgates42 https://github.com/ajgates42 Do you happen to have any
further reviewers you would suggest? Our list of information theory experts
is modest and it seems you are from this community. Any help here would
greatly aid the review process -- and therefore the work itself.

โ€”
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openjournals/joss-reviews/issues/1548?email_source=notifications&email_token=AGMT33MCZSNKLLCOMQKN743QAXFUVA5CNFSM4H6WCZTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2QIPNI#issuecomment-513836981,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGMT33LCTRZUHKCR4KPJ6HTQAXFUVANCNFSM4H6WCZTA
.

--
Alexander Gates
Postdoctoral Research Scholar
Center for Complex Network Research
Northeastern University
http://alexandergates.net

Hi Ryan (@autoplectic) -- You have been recommended as an expert in information theory; are you comfortable enough in C++ and/or python to confidently review this submission? And do you have the time an interest to do so?

Hi Georg (@georgmartius) -- You have also been recommended as an information theory expert. Same question as for Ryan: do you feel confident enough to review the submission, and do you have the time and interest to do so?

Hi Jack,
an even better qualified person would be Joseph Lizier http://lizier.me/joseph/. He wrote JIDT, a quite comprehensive java package.
I am afraid, I cannot invest the appropriate amount of time into it at the moment.

Hi Joseph (@jlizier) -- your colleague, Georg Martius, recommended you as an expert in information theory software packages. He mentioned that you worked primarily in Java -- would you feel confident reviewing a submission in C++ and Python? And, more importantly, do you have the time and interest to do so?

Hi @poulson - main challenge for me atm is time as I'm already loaded up with reviews over the next month. I could commit to early Sept, but I'm guessing that's a bit too long for you? If not, whilst my c++ is a bit rusty I think it would be ok for this

Thank you for the fast and candid response, Joseph (@jlizier)! I will try to move forward with finding other reviewers and -- if that turns out to run into substantial delays -- circle back with you in early September.

Hi Ryan (@autoplectic) -- Do you think you would be able to provide input on this submission?

:wave: Fabian (@fsteeg) -- Would you have the time and interest to provide feedback on this submission?

@poulson The package, website and the paper have been updated. Ready to review now. Hope we find the second reviewer soon. Thanks!

@whedon generate pdf

Attempting PDF compilation. Reticulating splines etc...

Fabian (@fsteeg) -- Would you have the time and interest to provide feedback on this submission?

Thanks for getting in touch @poulson, and sorry for the late reply, I was on vacation. My expertise is neither in information theory, nor C++ or Python, so I'm afraid I won't be able to contribute here.

Hi Ryan (@Autoplectic) -- Do you think you would be able to provide input on this submission?

Hi all, sorry for the delayed response. Yes, I can review it but it might take a couple weeks.

@whedon add @Autoplectic as reviewer

OK, @Autoplectic is now a reviewer

@whedon start review

OK, I've started the review over in https://github.com/openjournals/joss-reviews/issues/1609. Feel free to close this issue now!

Was this page helpful?
0 / 5 - 0 ratings