Skip to content
This repository has been archived by the owner on Jan 24, 2023. It is now read-only.

Add python3.10 support #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

john-dupuy
Copy link

Address #15

This is mostly just to fix https://github.com/google/earthengine-api for python3.10

@xmedeko @halvards can one of you take a look?

@google-cla
Copy link

google-cla bot commented Mar 4, 2022

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

For more information, open the CLA check for this pull request.

@kcaylor
Copy link

kcaylor commented May 10, 2022

Any update on this? Would be nice to get this approved and merged into the distribution...

@john-dupuy
Copy link
Author

Any update on this? Would be nice to get this approved and merged into the distribution...

No updates from my end, perhaps @halvards can take a look? I'm not sure who else has write access to this repo.
If this library is deprecated, it might be better to remove earthengine-api's dependence on it.

@kcaylor
Copy link

kcaylor commented May 11, 2022

@john-dupuy - I found a work-around. It turns out that ee.Initialize() allows passing in an http_transport argument that bypasses the problematic code in the httplib2shim library. So this works:

import httplib2
ee.Initialize(http_transport=httplib2.Http())

Whatever the httplib2shim code was doing, it seems that httplib2 works fine (so far). Agree this library should be removed from ee if possible.

@john-dupuy
Copy link
Author

john-dupuy commented May 11, 2022

@kcaylor thanks for the tip! Unfortunately, our project relies on the ee-cli tool, which uses the ee client that gets initialized here:
https://github.com/google/earthengine-api/blob/master/python/ee/cli/utils.py#L133-L139

However, maybe we can rework things to use the ee client directly rather than the cli tool.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants