-
-
Notifications
You must be signed in to change notification settings - Fork 36
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]: mmappickle: A Python 3 library to store memory mappable objects into pickle-compatible files #651
Comments
Hello human, I'm @whedon. I'm here to help you with some common editorial tasks. @glemaitre, 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:
|
|
@whedon generate pdf |
|
@clemaitre58 - please complete your review when you get a chance. |
@glemaitre - Any news from your side? I think I've addressed all the issues you mentioned, but I'm waiting for your feedback/confirmation to close them, and also to make the 1.0.1 release once your review is finished... I also think you could update the checkboxes in the checklist above, since a lot of these points have been addressed meanwhile. Thanks a lot! |
I'll do a pass at night.
…On 7 May 2018 at 15:11, Laurent Fasnacht ***@***.***> wrote:
@glemaitre <https://github.com/glemaitre> - Any news from your side? I
think I've addressed all the issues you mentioned, but I'm waiting for your
feedback/confirmation to close them, and also to make the 1.0.1 release
once your review is finished...
I also think you could update the checkboxes in the checklist above, since
a lot of these points have been addressed meanwhile.
Thanks a lot!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#651 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AHG9PxBKoTruG7mUoS0EYBNl9lx6ewKBks5twEf7gaJpZM4S8zRh>
.
--
Guillaume Lemaitre
INRIA Saclay - Parietal team
Center for Data Science Paris-Saclay
https://glemaitre.github.io/
|
@glemaitre - Great, thanks! |
@lfasnacht I think it could be nice to had a "getting started" section in your documentation in order to see quickly a basic example. |
Some links for the revision : |
@clemaitre58 After discussion with @glemaitre (UniNE-CHYN/mmappickle#9), it was decided to move the getting started to the front page of the documentation (http://mmappickle.readthedocs.io/en/latest/#quick-start), is it what you had in mind? |
I find the quick start on the landing page better than in the a separate page. |
@lfasnacht Ok for the quick start. |
@clemaitre58 It was hard, but now it's done. It is indeed much better, thank you very much for the suggestion. |
@glemaitre, @clemaitre58 - can you both confirm that we're good to accept here? |
good on my side
|
@lfasnacht - At this point could you make an archive of the reviewed software in Zenodo/figshare/other service and update this thread with the DOI of the archive? I can then move forward with accepting the submission. |
@whedon set 10.5281/zenodo.1291734 as archive |
I'm sorry @lfasnacht, I'm afraid I can't do that. That's something only editors are allowed to do. |
@arfon It seems that you have to do that ;-) Thanks a lot! |
@whedon set 10.5281/zenodo.1291734 as archive |
OK. 10.5281/zenodo.1291734 is the archive. |
@glemaitre, @clemaitre58 - many thanks for your reviews here ✨ @lfasnacht - your paper is now accepted into JOSS and your DOI is https://doi.org/10.21105/joss.00651 ⚡ 🚀 💥 |
🎉🎉🎉 Congratulations on your paper acceptance! 🎉🎉🎉 If you would like to include a link to your paper from your README use the following code snippet:
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:
|
Dear all, Thank you very much for your awesome work. I will definitely recommend submitting to JOSS to my colleagues, and will also consider submitting more of my work. Kind regards, |
Submitting author: @lfasnacht (Laurent Fasnacht)
Repository: https://github.com/UniNE-CHYN/mmappickle
Version: v1.0.0
Editor: @arfon
Reviewer: @glemaitre, @clemaitre58
Archive: 10.5281/zenodo.1291734
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
@glemaitre & @clemaitre58, 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 @arfon know.
Review checklist for @glemaitre
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 @clemaitre58
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: