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

CoastSat notebook: Add a line to specify a project ID for GEE #225

Closed
5 tasks done
2320sharon opened this issue Jan 26, 2024 · 2 comments
Closed
5 tasks done

CoastSat notebook: Add a line to specify a project ID for GEE #225

2320sharon opened this issue Jan 26, 2024 · 2 comments
Assignees
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers

Comments

@2320sharon
Copy link
Collaborator

2320sharon commented Jan 26, 2024

Add a line into the notebook that tells the user to enter the project ID in case the google auth method doesn't work.

import ee
# enter your project ID 
ee.Initialize(project='sample_id')

Tasks

  • Add the line to SDS_coastsat_classifier.ipynb
  • Make this line commented out
  • Add instructions on when to run this line
  • Show the user where they can find this info
  • Reference how geemap does this for reference
@2320sharon 2320sharon added bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers labels Jan 26, 2024
@2320sharon 2320sharon self-assigned this Jan 26, 2024
@2320sharon
Copy link
Collaborator Author

2320sharon commented Feb 9, 2024

Google brought back the old notebook way of authenticating later after deprecating it.

According to https://developers.google.com/earth-engine/guides/auth
The default auth mode will be :

notebook if running in other non-Colab Jupyter notebooks
localhost if a web browser is detected and no gcloud binary is installed

This means in our notebooks the default method will be 'notebook' and we should only tell the user to specify the project id if the selected project does not have earth engine enabled.

The full changelog of changes to earth engine is here https://developers.google.com/earth-engine/changelog#python_api_v01383

  • Update the minimum version of earth-engine required in the pyproject.toml to be greater than 0.1.388
  • Allow the user to pass auth_mode = 'notebook'
  • Run auth with notebook by default and localhost as a backup method
  • Tell the user when they need to enter their project id and provide default code
  • Link the user to a tutorial on how to get the project id https://developers.google.com/earth-engine/cloud/earthengine_cloud_project_setup
  • Bring back the old gif with notebook

@2320sharon
Copy link
Collaborator Author

These changes have been merged into main and will be added in the next official release.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant