Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign up[PRE REVIEW]: pyHoops: A Python package for advanced basketball data analytics #1784
Comments
This comment has been minimized.
This comment has been minimized.
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:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
What happens now? This submission is currently in a 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. |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
Failed to discover a valid open source license. |
This comment has been minimized.
This comment has been minimized.
PDF failed to compile for issue #1784 with the following error: /app/vendor/bundle/ruby/2.4.0/bundler/gems/whedon-efe915e61673/lib/whedon.rb:135:in |
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Oct 4, 2019
Hello @alessandroBombelli, thanks for your interest in JOSS. Before we proceed, during my review of your submission, I could not find the research application of your software. While useful for folks in basketball, I don't quite see where someone would cite this in the academic literature (not an explicit requirement, but one way we think about whether software has a research use). Can you clarify this, per one of our submission requirements?
|
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Oct 4, 2019
alessandroBombelli/pyHoops#46 should fix the immediate paper building process |
This comment has been minimized.
This comment has been minimized.
alessandroBombelli
commented
Oct 4, 2019
Dear @kyleniemeyer, apologies if the research application is not clear. I agree that, at a first glance, the obvious research application might not be as straight-forward as in other cases. On the other hand, a strong interest in data analytics applied to sports has been growing in recent years, as testified by ad-hoc conferences (MIT Sloan Sports Analytics Conference) and journals (Journal of Sports Sciences). To this avail, the research application of this work is the possibility to compute a set of team performance indices (per player and per lineup) to be used, as example, to infer lineup performances in future games. Knowing how different lineups perform against different opposing lineups (e.g., "tall", or "fast" lineups), can be used as part of a decision-making process to improve performances of a team. In addition, a sufficiently comprehensive database retrieved via pyHoops (e.g., a full season), can be used as the input for a prediction model (e.g., random forest) to assess what are the features (lineup- or player-specific) more crucial for a team's success. I hope this answers your question. I am willing to elaborate more if the answer is not satisfactory. Thank you! |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Oct 4, 2019
I can take |
This comment has been minimized.
This comment has been minimized.
alessandroBombelli
commented
Oct 4, 2019
Dear @danielskatz, sorry for the issue. but it is my first time here and, as such, I am not familiar with the formatting. I followed the example provided in the guidelines but thought that, being a single author, the distinction between affiliation and affiliations was not necessary. I fixed the issue now. Please let me know otherwise. |
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Oct 4, 2019
@danielskatz thanks for volunteering—did you mean to close the issue? |
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Oct 4, 2019
@alessandroBombelli ok, so I think it would help to revise your article to better describe the research applications of the software, so this is clear to reviewers (and readers), focusing specifically on the types of research questions/problems this helps answer. What you are describing to me sounds more like what a basketball team might use, but not an academic researcher writing a publication (for example)—but I may still be misunderstanding. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Oct 4, 2019
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Oct 4, 2019
I've suggested more changes in alessandroBombelli/pyHoops#48 - after merging this, you can enter |
This comment has been minimized.
This comment has been minimized.
kyleniemeyer
commented
Oct 4, 2019
@whedon assign @danielskatz as editor |
This comment has been minimized.
This comment has been minimized.
OK, the editor is @danielskatz |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Oct 7, 2019
@alessandroBombelli - do you have suggestions for reviewers? They can be from this list of potential reviewers (please start at the bottom of the list), or individuals who are not on this list; if so, please mention their GitHub handles here. |
This comment has been minimized.
This comment has been minimized.
alessandroBombelli
commented
Oct 8, 2019
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
alessandroBombelli
commented
Oct 8, 2019
@danielskatz, I slightly modified references because I realized two of them were not correctly compiled. In one case, there was a @ missing. In the other case, where I am citing a website, I am not sure what the problem is. I am using the same citation type I would use in a classic LaTeX file, where I know it should work. On the other hand, I am not very familiar with the integration of the .md file with the .bib file, so I have little control on what is going wrong. Could you please help me with that? Thanks! |
This comment has been minimized.
This comment has been minimized.
alessandroBombelli
commented
Oct 8, 2019
@danielskatz never mind, it is fixed now. Sorry for the confusion |
This comment has been minimized.
This comment has been minimized.
alessandroBombelli
commented
Oct 8, 2019
Dear @danielskatz, I checked the list and I think @mado89 should be a good match. No preference for the second reviewer instead. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Oct 8, 2019
Perhaps you can suggest some other people not on the list? Maybe from other types of sport analytics? |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Oct 8, 2019
Also, for |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Oct 8, 2019
|
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Oct 8, 2019
If not, could you suggest some others in the field of software and sports analytics who might be suitable? |
This comment has been minimized.
This comment has been minimized.
@danielskatz I can do it. But I might need some time (and maybe a gentle reminder in 1.5 weeks) as I am quite busy this and next week. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Oct 8, 2019
Thanks @mado89 - we won't start until I find another reviewer, but I'll put you into the system for now, and I will be sure to remind you again once things start, or it's been a little time :) |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Oct 8, 2019
This comment has been minimized.
This comment has been minimized.
OK, the reviewer is @mado89 |
This comment has been minimized.
This comment has been minimized.
alessandroBombelli
commented
Oct 8, 2019
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
PDF failed to compile for issue #1784 with the following error: Error reading bibliography ./pyHoops_references.bib (line 18, column 3): |
This comment has been minimized.
This comment has been minimized.
alessandroBombelli
commented
Oct 8, 2019
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
alessandroBombelli
commented
Oct 8, 2019
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
alessandroBombelli
commented
Oct 8, 2019
@danielskatz done! |
whedon commentedOct 4, 2019
•
edited
Submitting author: @alessandroBombelli (Alessandro Bombelli)
Repository: https://github.com/alessandroBombelli/pyHoops
Version: v1.5
Editor: @danielskatz
Reviewers: @mado89
Author instructions
Thanks for submitting your paper to JOSS @alessandroBombelli. Currently, there isn't an JOSS editor assigned to your paper.
@alessandroBombelli 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: