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

[SEDONA-419] Remove SedonaKepler and SedonaPyDeck from sedona.spark import * #1074

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

jiayuasu
Copy link
Member

@jiayuasu jiayuasu commented Nov 7, 2023

Did you read the Contributor Guide?

Is this PR related to a JIRA ticket?

What changes were proposed in this PR?

Remove SedonaKepler and SedonaPyDeck from the __init__.py file.

Users who use from sedona.spark import * to do SedonaKepler/SedonaPyDeck visualization will need to explicitly import

from sedona.maps.SedonaKepler import SedonaKepler
from sedona.maps.SedonaPyDeck import SedonaPyDeck

How was this patch tested?

Passed all unit tests

Did this PR include necessary documentation updates?

  • No, this PR does not affect any public API so no need to change the docs.

@jiayuasu jiayuasu added this to the sedona-1.5.1 milestone Nov 7, 2023
@jiayuasu jiayuasu merged commit 4c5fa83 into master Nov 7, 2023
30 of 40 checks passed
@jiayuasu jiayuasu deleted the isolate-pydeck-keplergl-import branch November 7, 2023 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant