Skip to content
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

Adding cartoee module to geemap #161

Merged
merged 4 commits into from
Oct 27, 2020
Merged

Adding cartoee module to geemap #161

merged 4 commits into from
Oct 27, 2020

Conversation

KMarkert
Copy link
Member

PR to merge functionality from cartoee into geemap as a module. Users will need to explicitly import module (i.e. from geemap import catoee) to prevent users from having to install cartoee dependencies without use. Module checks for installed dependencies on import and installs if not available for ease of use.

Added files:

  • geemap/cartoee.py: cartoee module to create publication quality figures with EE data
  • examples/notebooks/cartoee_quickstart.ipynb: quickstart notebook highlighting basic functionality and use of the module
  • examples/notebooks/cartoee_projections.ipynb: notebook highlighting different map types and result projection caveats when using cartoee

@KMarkert
Copy link
Member Author

Should be noted that install messages are based on logging. So if logging level is not set, nothing will be printed. To get information on checking dependencies one would need to set logging level to INFO.

Here is an example with expected output:

In [1]: import logging
In [2]: logging.basicConfig(level=logging.INFO)
In [3]: from geemap import cartoee
INFO:root:All dependencies are successfully imported/installed!

@giswqs
Copy link
Member

giswqs commented Oct 27, 2020

Worked like a charm. Thanks a lot for your contribution! #157

@giswqs giswqs merged commit 5c1f529 into master Oct 27, 2020
@giswqs giswqs deleted the feature/cartoee branch October 27, 2020 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants