Skip to content

Conversation

@adamjstewart
Copy link

When using ee in Python, mypy complains with:

error: Name "ee.Image" is not defined  [name-defined]
error: Name "ee.ImageCollection" is not defined  [name-defined]
error: Module "ee" does not explicitly export attribute "ImageCollection"  [attr-defined]
error: Module "ee" does not explicitly export attribute "Filter"  [attr-defined]
error: Module "ee" does not explicitly export attribute "Filter"  [attr-defined]
error: Name "ee.ImageCollection" is not defined  [name-defined]
error: Name "ee.ImageCollection" is not defined  [name-defined]
error: Module "ee" does not explicitly export attribute "Filter"  [attr-defined]
error: Module "ee" does not explicitly export attribute "Filter"  [attr-defined]
error: Module "ee" does not explicitly export attribute "Filter"  [attr-defined]
error: Module "ee" does not explicitly export attribute "Geometry"  [attr-defined]
error: Module "ee" does not explicitly export attribute "EEException"  [attr-defined]
error: Name "ee.ImageCollection" is not defined  [name-defined]
error: Module "ee" does not explicitly export attribute "Geometry"  [attr-defined]
error: Name "ee.ImageCollection" is not defined  [name-defined]

This is because none of these classes are explicitly exported via __all__. This PR fixes that.

copybara-service bot pushed a commit that referenced this pull request Dec 9, 2025
PiperOrigin-RevId: 842221132
copybara-service bot pushed a commit that referenced this pull request Dec 9, 2025
PiperOrigin-RevId: 842221132
@adamjstewart
Copy link
Author

@schwehr anything else needed before this PR can be merged?

@adamjstewart
Copy link
Author

Trying to close out old PRs. Is this something that is wanted?

@schwehr
Copy link
Contributor

schwehr commented Jan 13, 2026

I think it should go into the interface, but there is a worry about unintentional changes to the interface. Please put a typing.TYPE_CHECKING guard around the change.

Signed-off-by: Adam J. Stewart <[email protected]>
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