Join GitHub today
GitHub is home to over 36 million developers working together to host and review code, manage projects, and build software together.
Sign up[REVIEW]: FSharpGephiStreamer: An idiomatic bridge between F# and network visualization #1445
Comments
whedon
assigned
danielskatz
May 13, 2019
whedon
added
the
review
label
May 13, 2019
whedon
referenced this issue
May 13, 2019
Closed
[PRE REVIEW]: FSharpGephiStreamer: An idiomatic bridge between F# and network visualization #1214
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. @cgravill, it looks like you're currently assigned as the reviewer for this paper If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/openjournals/joss-reviews) repository. As a reviewer, you're probably currently watching this repository which means for GitHub's default behaviour you will receive notifications (emails) for all reviews To fix this do the following two things:
For a list of things I can do to help you, just type:
|
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
danielskatz
assigned
cgravill
May 13, 2019
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
May 13, 2019
Please carry out your review in this issue by updating your checklist above. The checklist gives me some insight into the review process and any blocking issues, as well as being the full set of items that once passed will lead us to accept the submission. And please fully read the the top 2 comments in this issue. If you have any problems or questions, please feel free to let me know. |
danielskatz
unassigned
cgravill and
danielskatz
May 13, 2019
whedon
assigned
cgravill,
danielskatz and
PrashantVaidyanathan
May 14, 2019
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
May 18, 2019
If you can at least check your "Conflict of interest" and "Code of Conduct" boxes when you feel ready, that would help me know that the processes are working and that you are starting :) (And @PrashantVaidyanathan, yes, I do remember that you will not be working on this 'till next week due to travel) |
This comment has been minimized.
This comment has been minimized.
Sure, completed those steps @danielskatz How precisely should these requirements be taken? For example, they have a plain text license but it includes a .txt extension. Is that OK? https://joss.readthedocs.io/en/latest/review_criteria.html#software-license |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
May 18, 2019
You could ask that this be changed to have no extension, since the JOSS criteria says "Does the repository contain a plain-text LICENSE file with the contents of an OSI approved software license?" and since the GitHub guidance for this is for it to either have no extension or be .md. You could open an issue in the software repo to request this, and put in a comment here pointing to that issue. |
This comment has been minimized.
This comment has been minimized.
I'm not aware it's a conflict of interest but for full transparency:
|
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
May 18, 2019
Are you both employed in the same group? Do you work together on the same projects?
Expertise in the subject is good. With both factors in mind, do you feel that you can provide a fair and unbiased review and advise? |
This comment has been minimized.
This comment has been minimized.
Yes, we're both employed in this group https://www.microsoft.com/en-us/research/group/biological-computation/ and have overlap in projects. We've not discussed the contents of this work. I feel I'm able to give a fair and unbiased review. |
May 18, 2019
This was referenced
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
May 19, 2019
Thanks for providing this info, @cgravill Given the particular circumstances of this software and this conflict, we (the associate editors-in-chief) have decided that we will note it and waive it for this review. So please continue the review. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
May 23, 2019
|
This comment has been minimized.
This comment has been minimized.
@danielskatz Apologies for the delay. I had spotty net connection over the last few days while travelling. I've started reviewing the submission now. |
This comment has been minimized.
This comment has been minimized.
danielskatz
commented
Jun 3, 2019
It looks like there are some issues being worked through, which is good - @muehlhaus and @cgravill and @PrashantVaidyanathan, please let me know if anything seems to get stuck |
whedon commentedMay 13, 2019
•
edited by PrashantVaidyanathan
Submitting author: @muehlhaus (Timo Mühlhaus)
Repository: https://github.com/CSBiology/FSharpGephiStreamer
Version: v1.1.0
Editor: @danielskatz
Reviewer: @cgravill, @PrashantVaidyanathan
Archive: Pending
Status
Status badge code:
Reviewers and authors:
Please avoid lengthy details of difficulties in the review thread. Instead, please create a new issue in the target repository and link to those issues (especially acceptance-blockers) by leaving comments in the review thread below. (For completists: if the target issue tracker is also on GitHub, linking the review thread in the issue or vice versa will create corresponding breadcrumb trails in the link target.)
Reviewer instructions & questions
@cgravill & @PrashantVaidyanathan, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @danielskatz know.
Review checklist for @cgravill
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?Review checklist for @PrashantVaidyanathan
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?