-
-
Notifications
You must be signed in to change notification settings - Fork 38
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
[REVIEW]: A reusable tree-based web-visualization to browse EDAM ontology, and contribute to it. #698
Comments
Hello human, I'm @whedon. I'm here to help you with some common editorial tasks. @alexgarciac it looks like you're currently assigned as the reviewer for this paper 🎉. ⭐ Important ⭐ 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:
|
|
PDF failed to compile for issue #698 with the following error: % Total % Received % Xferd Average Speed Time Time Time Current 0 0 0 0 0 0 0 0 --:--:-- --:--:-- --:--:-- 0 |
Dear @bryan-brancotte, sorry for the delay. Mostly because someone stopped communicating - hard to deal with that. But we'll move on now. To expedite the work can you add the bibliography and compile the PDF until it works? Also please go through above check list and ascertain the reviewer can check the boxes. Much faster if you make sure we can tick them than wait for a back and forth. Meanwile @alexgarciac can you go through the JOSS tips for review at http://joss.theoj.org/about#reviewer_guidelines. Thanks! |
Hi @pjotrp, thanks a lot for handling this. Regarding the bibliography, the compilation error seems weird, because the bib file is actually there as far as I can tell (https://github.com/IFB-ElixirFr/edam-browser/blob/master/paper_resources/paper.bib). On the previous round @arfon actually provided the pdf manually. Is there something we should correct in the markdown or the bibtex to avoid the compilation errors? Thanks again! |
@hmenager - the issue here is that there are two papers so @whedon doesn't know what to do:
I've compiled this locally and uploaded here: 10.21105.joss.00698.pdf |
The second paper should be renamed? |
Yes. The regex goes looking for anything with |
@hmenager do you mind confirming we should be able to check all relevant boxes? |
@pjotrp I will check that carefully with @bryan-brancotte, and get back to you real soon. From quickly checking the list above, I think we miss a release and the contribution guidelines. We'll ping you as soon as these are done. |
Excellent. |
Hi @pjotrp, we renamed the long_paper.md and reviewed all the boxes: they should be good to check. |
Note that CI is pending on validation from https://github.com/IFB-ElixirFr/edam-browser/issues/15 |
@alexgarciac please start the review by ticking the boxes at the top of this page. |
@pjotrp I have ticked the boxes but, a naive question: where do I hit submit? |
@pjotrp also, there are typos and issues wrt the language, how do I send these to the authors? |
@alexgarciac you can report on the projects issue tracker (when relevant) and here. This issue tracker we use to do the actual public review. There is no submit button :). Just respond here. |
Hi, I have checked the appropriate boxes. the ones not checked have comments in the issue tracker of the edam prj. |
Other review comments (also made over the EDAM issue tracker). the symbol "->" indicates each issue reported in the EDAM issue tracker. -> Please add DOIs to the references where applicable. For instance, @Article{hoehndorf2015aber, Same is true for other references. -> There are issues with the language. Please have this paper proof read by a native speaker. there are parts in the manuscript that requiere some attention. For instance To facilitate these suggestions, the EDAM Browser lets users access a form letting them propose changes at any point of their exploration. You could say something like: In order to make it easy for the community to submit comments to the ontology editors, we have implemented a form that facilitates this communication. the section "easy of community..." should be just "Facilitating community feedback" The five lines in this section need some work -less is more here. At: Please be more concise, remove unnecessary adjectives such as "paramount". Focus on what this onto is for and how your software is addressing some issue wrt the development process. In this case, you are i) facilitating the navigation and ii) making is easy for the community to communicate issues to the ontology developers via a simple form.
In order to present the functionality of the software consider building a user story that brings together the problem from the perspective of the user; your software is in your own words "This browser is tailored to the needs of EDAM users that might not be ontology experts" so just build the user story from this perspective. there is also another perspective; that somewhat solved by the communication channel you are bridging between the community and the ontology developers via a form. This should also be part of the user story. Then simply state how your software is solving these stories. Use this part in order to present most, if not all, the functionalities of your software. -> Please look at Its interface is not designed to be a generic ontology navigation and edition platform, a goal already achieved by many other systems (AberOWL(Hoehndorf et al. 2015), BioPortal(Whetzel et al. 2011), OLS - Ontology Lookup Service(Jupp et al. 2015), Ontobee(Xiang et al. 2011), WebProtégé(Tudorache et al. 2013)). I presume the "its" refers to the EDAM interface. Just use the form "the edam interface....." Please be mindful here because BioPortal is not an ontology editor, it is an ontology publication platform. You could edit the ontology by hand if you want to. By the same token, Ontobee is not an ontology editor. Just organize this part of your paper making sure that you are distinguishing a publication platform/ontology repository vs an ontology editor. Also, WebProtégé(Tudorache et al. 2013)), why do u have )) ? One more, leave a space between the name of the software and the parenthesis indicating the reference -please check this everywhere in the manuscript. -> Please in the manuscript, as well as in the readme for the git make sure that u clearly address the following issues: 1- This one requires attention more in the manuscript than in the readme. As you state that your form is meant to support the evolution of the ontology then there is a community component. are comments going to be visible for everyone to see and keep commenting on? is this commenting feature just a form by means of which issues go straight into a git issue tracker? Below some more specifics on this point. Community guidelines: Are there clear guidelines for third parties wishing to 1) Contribute to the software 2) Report issues or problems with the software 3) Seek support 2- This one is specific for the manuscript and also requieres that you update your bib file. References: Do all archival references that should have a DOI list one (e.g., papers, datasets, software)? 3- this one is more related to the manuscript because IMHO your software development is easy to follow. However, a visit to the documentation in the actual code and overall project would not harm. In the manuscript you need to better describe the functionality of the software. the software is very simple, this is good, but it is poorly described in the paper. You may want to relate the description of the functionality to specific use cases. Functionality documentation: Is the core functionality of the software documented to a satisfactory level (e.g., API method documentation)? 4- Installation instructions: Is there a clearly-stated list of dependencies? Ideally these should be handled with an automated package management solution. I could not find the list of dependencies. |
Thans @alexgarciac for your very thorough review. @bryan-brancotte I think it is a good idea to look at all suggestions and see if you can improve the paper. The goal is to present your work in the best way so people start using it! I don't think it is a lot of work, so it is close to being accepted with minor revisions only. Of the missing check boxes, I think only the one on community guidelines to be really important. See if we can check that box. Also DOIs and installation instructions would be a good idea. |
ping @bryan-brancotte |
Hi @pjotrp |
OK, no problem. As long as we know about it we don't worry. |
@whedon generate pdf |
|
@whedon generate pdf |
@whedon generate pdf |
|
@whedon generate pdf |
|
@whedon generate pdf |
|
Hello @pjotrp and @alexgarciac, we have now finished revising the paper, based on the suggestions you made @alexgarciac. Thank you so much for all your comments, I hope the new version of the manuscript adresses all of them. |
@alexgarciac are you happy to release this submission into the public domain? |
ping @alexgarciac |
Hi, I have read the latest version and it looks ready to go. @pjotrp answering your question: yes, ready to go into the public domain. |
all check boxes (the pertinent ones) on the issue tracker have been checked. |
Thank you @alexgarciac! 👍 Thank you @bryan-brancotte for your great contribution, there should be more linked data contributions like these. The review process is now complete. To finalize your submission and accept your paper in JOSS, we need two things. First, can you confirm that all references in your bibliography have a DOI (if one exists). Second, we need you to deposit a copy of your software repository (including any revisions made during the JOSS review process) with a data-archiving service. To do so:
|
Thanks @alexgarciac @pjotrp for making this contribution being published ! |
@whedon set 10.5281/zenodo.1314288 as archive |
OK. 10.5281/zenodo.1314288 is the archive. |
@arfon we are ready to accept this paper. @bryan-brancotte note that arfon is still on holiday, so it may take a little while. |
no problem, thanks |
@alexgarciac - many thanks for your review here and to @pjotrp for editing this submission ✨ @bryan-brancotte - your paper is now accepted into JOSS and your DOI is: https://doi.org/10.21105/joss.00698 ⚡️:rocket: :boom: |
🎉🎉🎉 Congratulations on your paper acceptance! 🎉🎉🎉 If you would like to include a link to your paper from your README use the following code snippets:
This is how it will look in your documentation: We need your help! Journal of Open Source Software is a community-run journal and relies upon volunteer effort. If you'd like to support us please consider doing either one (or both) of the the following:
|
Submitting author: @bryan-brancotte (Bryan Brancotte)
Repository: https://github.com/IFB-ElixirFr/edam-browser
Version: v1.0.0
Editor: @pjotrp
Reviewer: @alexgarciac
Archive: 10.5281/zenodo.1314288
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) 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
@alexgarciac, 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.theoj.org/about#reviewer_guidelines. Any questions/concerns please let @pjotrp know.
Review checklist for @alexgarciac
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
paper.md
file include a list of authors with their affiliations?The text was updated successfully, but these errors were encountered: