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[REVIEW]: Adeft: Acromine-based Disambiguation of Entities from Text with applications to the biomedical literature #1708
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. @GullyAPCBurns, @gbader it looks like you're currently assigned to review 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:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
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.
@GullyAPCBurns @gbader, please go ahead and review away ;) |
This comment has been minimized.
This comment has been minimized.
@GullyAPCBurns @gbader, if you have doubts about how to proceed, please ask! |
This comment has been minimized.
This comment has been minimized.
Hello. I completed the review. The paper and software look good. My graduate student @JohnGiorgi helped review the software. We were able to get it to work and it successfully disambiguated “ER” (endoplasmic reticulum) in some test text. |
This comment has been minimized.
This comment has been minimized.
Great, thanks @gbader! I've noticed that you left out "State of the field" in your review, @johnbachman, could you please address that in the manuscript? @GullyAPCBurns, let us know if you have any difficulties with the review process, I'm here to assist! |
This comment has been minimized.
This comment has been minimized.
@brainstorm Will do! |
This comment has been minimized.
This comment has been minimized.
@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.
@brainstorm @gbader We added a new paragraph to the manuscript briefly describing where Adeft fits in the context of word sense disambiguation more generally. |
This comment has been minimized.
This comment has been minimized.
Thanks @johnbachman! @GullyAPCBurns, please let me know if you have any doubts about the review process? |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
The new paragraph looks great. In my view, this paper should be accepted. Sorry for the delay. I like the idea of this journal, but one issue with running peer review on github is that the emails all come from github, which I happen to have filtered due to the large number of automated emails I get from github. |
This comment has been minimized.
This comment has been minimized.
Thanks @gbader for your input, we are still waiting for @GullyAPCBurns... shall we regard him as missing reviewer at this point as he's not replying? |
This comment has been minimized.
This comment has been minimized.
@brainstorm — this submission has been silent for a while. It looks like you have an MIA reviewer. I suppose you did try to reach them by other means? (email, Twitter) It may be time to take some executive decision here on finding a replacement reviewer. |
This comment has been minimized.
This comment has been minimized.
I contacted @GullyAPCBurns by email on Dec 13th, he said he would be able to review. |
This comment has been minimized.
This comment has been minimized.
Sincere apologies for having been unresponsive. I will recommit to completing this review as quickly as possible in the next two weeks. |
This comment has been minimized.
This comment has been minimized.
Here are some things you can ask me to do:
|
This comment has been minimized.
This comment has been minimized.
@danielskatz, I can't seem to check any of the boxes on the above form. Am I missing something very obvious? |
This comment has been minimized.
This comment has been minimized.
Please see the first comment - did you accept the invite as mentioned in step 2 there? |
This comment has been minimized.
This comment has been minimized.
Got it. Confirmed and yet more apologies for the delay. |
This comment has been minimized.
This comment has been minimized.
I was able to review the paper and software. The code looks great and the documentation is exemplary. I ran the available Jupyter notebooks and was able to disambiguate 'Gut Cancer (GC)' effectively. Again, I apologize for not having done this immediately. I'd recommend that you proceed with publication 'as is'. |
This comment has been minimized.
This comment has been minimized.
Great, thanks @GullyAPCBurns! |
This comment has been minimized.
This comment has been minimized.
@whedon check references from branch joss-paper |
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.
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Looks good to my too, @johnbachman, next step is for you to deposit it on Zenodo and we'll archive it in JOSS, let me know if you run into any problems with that. Pretty close to publication now ;) |
This comment has been minimized.
This comment has been minimized.
Great, will do, thanks to all! |
This comment has been minimized.
This comment has been minimized.
(Edited with correct info) Hi @brainstorm, the release for the paper is up at Zenodo here: https://zenodo.org/record/3608243 The version is 0.5.3, DOI is 10.5281/zenodo.3608243 Also, a question--should we add in DOIs for the 7 listed as "missing"? |
This comment has been minimized.
This comment has been minimized.
@whedon generate pdf |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@brainstorm Sorry, updated the comment above with correct DOI information |
This comment has been minimized.
This comment has been minimized.
@whedon set https://zenodo.org/record/3608243 as archive |
This comment has been minimized.
This comment has been minimized.
https://zenodo.org/record/3608243 doesn't look like an archive DOI. |
This comment has been minimized.
This comment has been minimized.
@whedon set 10.5281/zenodo.3608243 as archive |
This comment has been minimized.
This comment has been minimized.
OK. 10.5281/zenodo.3608243 is the archive. |
This comment has been minimized.
This comment has been minimized.
No worries John, gotcha. @danielskatz LGTM, can you find any irregularity pre-publishing it for good? |
This comment has been minimized.
This comment has been minimized.
|
whedon commentedSep 5, 2019
•
edited
Submitting author: @johnbachman (John Bachman)
Repository: https://github.com/indralab/adeft
Version: 0.5.3
Editor: @brainstorm
Reviewer: @GullyAPCBurns, @gbader
Archive: 10.5281/zenodo.3608243
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
@GullyAPCBurns & @gbader, 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 @brainstorm know.
Review checklist for @GullyAPCBurns
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @gbader
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper