Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign up[REVIEW]: geemap: A Python package for interactive mapping with Google Earth Engine #2305
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @fbiljecki, @steflhermitte it looks like you're currently assigned to review this paper Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post. 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:
|
|
👋🏼 @fbiljecki @steflhermitte this is the review thread for the paper. All of our communications will happen here from now on. Both reviewers have checklists at the top of this thread with the JOSS requirements. As you go over the submission, please check any items that you feel have been satisfied. There are also links to the JOSS reviewer guidelines. The JOSS review is different from most other journals. Our goal is to work with the authors to help them meet our criteria instead of merely passing judgment on the submission. As such, the reviewers are encouraged to submit issues and pull requests on the software repository. When doing so, please mention We aim for reviews to be completed within about 2-4 weeks. Please let me know if any of you require some more time. We can also use Whedon (our bot) to set automatic reminders if you know you'll be away for a known period of time. It's for both of you the first review here, so it can be a bit overwhelming. Please feel free to ping me (@hugoledoux) if you have any questions/concerns. |
@fbiljecki I see you ticked several checkboxes above, but not all. If you haven't done some yet it's fine, but if you have questions or think the software/docs should be modified, the idea is that you state so here in a new comment. If it's something complex you can open a new issue in the repo of the project and put a link here too. |
@fbiljecki Thank you very much for taking the time to review the repo! Regarding the two unchecked items:
I would be happy to make improvement if needed. Thank you. |
Thanks @giswqs, nice job on the software. |
@fbiljecki Thank you very much for your time. Appreciated. |
Dear @giswqs, First I would like to thank you for the development of what looks at first sight/experience an extremely useful, powerful and well-designed package for interacting with GEE. Looking forward to explore it further in the coming days when finishing the review. When installing the package I faced some initial glitches as Hope to finish the rest of the review in the coming hours/days. Groet, |
Here are some things you can ask me to do:
|
@hugoledoux Practical question: how do I tick the review checklist? I see the checkboxes, but can't tick them. |
@steflhermitte Thank you very much for taking to time to test the package. The ipyleaflet dependency sometime can be a bit tricky to install, as it might conflict with some of you existing packages. The best way to install geemap is to create a fresh conda environment, if you have Anaconda or Miniconda installed on your computer. I also have a YouTube video showing users how to install the package using conda. Hope it helps. Thanks. https://github.com/giswqs/geemap#installation
|
You might have received an invitation email earlier on. You will need to click the link in that email to accept the invitation. After that, you should be able to tick the review checklist. |
@giswqs Thanks for the clarifications! I agree that your manual is very clear. I was just testing it in my own environment and that seems to work (after solving the conflicting dependencies). The invitation seems not to work any longer |
@whedon re-invite @steflhermitte as reviewer |
OK, the reviewer has been re-invited. @steflhermitte please accept the invite by clicking this link: https://github.com/openjournals/joss-reviews/invitations |
Dear @giswqs and @hugoledoux, I have been reviewing the software and accompanying paper and want to congratulate @giswqs on the nice piece of software. Geemap contains very useful tools for the designed audience that want to use Google Earth Engine in a Jupyter Notebook environment. The tools are simple, perform good and are well documented by example notebooks and accompanying Youtube videos. Based on this review I recommend the acceptance of this submission. Best regards, |
@steflhermitte Thank you very much for your support! Appreciated. |
Nice work @giswqs, this review was very smooth! The 2 reviewers now recommend acceptance so we're reaching the end of the review! At this point could you:
Please list the DOI of the archived version here. I can then move forward with accepting the submission. |
Dear @hugoledoux, I have made the requested changes. Thanks.
|
i have installed geemap through Anaconda, but when I coding in Jupiter notebook, I cannot open the map as you shown in video, what would be the wrong? |
@FainazInamdeen Did you create a fresh conda env to install geemap? Check the installation guidelines. If you still encounter issues, please open a new issue here. This thread is for the joss-review paper only, not for trouble shooting package installation issues.
|
@qiusheng Wu
Sorry didn't notice, btw I have sorted the issue. I think it was because
jupyter notebook before I opened through IE after I changed it into google
chrome, now its working. Thank you for an amazing tutorial.
…On Sat, Jul 4, 2020 at 10:38 PM Qiusheng Wu ***@***.***> wrote:
@FainazInamdeen <https://github.com/FainazInamdeen> Did you create a
fresh conda env to install geemap? Check the installation guidelines
<https://github.com/giswqs/geemap#installation>. If you still encounter
issues, please open a new issue here
<https://github.com/giswqs/geemap/issues>. This thread is for the
joss-review paper only, not for trouble shooting package installation
issues.
conda create -n gee python=3.7
conda activate gee
conda install mamba -c conda-forge
mamba install geemap -c conda-forge
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2305 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AQEBH4MNZKKBHK3OOFOV4OLRZ6HKVANCNFSM4NYKDDFQ>
.
|
@whedon generate pdf |
@whedon set 10.6084/m9.figshare.12608834.v1 as archive |
OK. 10.6084/m9.figshare.12608834.v1 is the archive. |
@whedon set v0.7.6 as version |
OK. v0.7.6 is the version. |
@whedon accept |
|
|
Check final proof If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#1546, then you can now move forward with accepting the submission by compiling again with the flag
|
Congratulations @giswqs your submission is now accepted! Your paper will be handled and published in the near future. Thanks to @fbiljecki and @steflhermitte for the reviews and feedback! |
@hugoledoux @fbiljecki @steflhermitte Thank you very much for taking the time to review the package. Much appreciated. |
@whedon @hugoledoux Is there anything I need to do on my end? It seems the paper is not moving forward. |
I'm sorry human, I don't understand that. You can see what commands I support by typing:
|
@giswqs no you don't, it should be picked up by one of the journal editor. It's summer I guess, sorry for delay |
@giswqs - sorry this slipped by - I'll work on now |
@giswqs - Please merge the changes in giswqs/geemap#115 then we can proceed to publication |
@danielskatz I have merged your pull request. Thank you very much for your help! |
@whedon accept |
|
|
Check final proof If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#1573, then you can now move forward with accepting the submission by compiling again with the flag
|
@whedon accept deposit=true |
|
|
Here's what you must now do:
Any issues? Notify your editorial technical team... |
Thanks to @fbiljecki & @steflhermitte for reviewing and @hugoledoux for editing! Congratulations to @giswqs (Qiusheng Wu)!! |
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:
|
whedon commentedJun 8, 2020
•
edited
Submitting author: @giswqs (Qiusheng Wu)
Repository: https://github.com/giswqs/geemap
Version: v0.7.6
Editor: @hugoledoux
Reviewer: @fbiljecki, @steflhermitte
Archive: 10.6084/m9.figshare.12608834.v1
Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.
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
@fbiljecki & @steflhermitte, 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 @hugoledoux know.
Review checklist for @fbiljecki
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @steflhermitte
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper