Skip to content

Remove st.deck_gl_chart#1950

Merged
kantuni merged 13 commits intostreamlit:developfrom
kantuni:remove/st-deck-gl-chart
Sep 17, 2020
Merged

Remove st.deck_gl_chart#1950
kantuni merged 13 commits intostreamlit:developfrom
kantuni:remove/st-deck-gl-chart

Conversation

@kantuni
Copy link
Copy Markdown
Collaborator

@kantuni kantuni commented Sep 1, 2020

Closes #1457.

@kantuni kantuni requested a review from randyzwitch as a code owner September 1, 2020 12:36
@kantuni kantuni requested a review from a team September 1, 2020 12:36
Copy link
Copy Markdown
Contributor

@randyzwitch randyzwitch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doc changes LGTM

@kantuni kantuni added type:docs Requests for changes to docs (will be rerouted to docs repo) and removed type:docs Requests for changes to docs (will be rerouted to docs repo) labels Sep 1, 2020
Copy link
Copy Markdown
Contributor

@akrolsmir akrolsmir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Just one small place to replace "deck_gl_chart" with "pydeck_chart".

_create_option(
"mapbox.token",
description="""Configure Streamlit to use a custom Mapbox
token for elements like st.deck_gl_chart and st.map.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replace with "like st.pydeck_chart"?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eagle eye ;)

@karriebear
Copy link
Copy Markdown
Contributor

Do we need to remove the enqueue + marshaling from lib/streamlit/elements/deck_gl.py? Not sure how this works with the deprecation label added as part of #1943 if we also removed it from docs/api

@kantuni kantuni merged commit 8be8a1b into streamlit:develop Sep 17, 2020
@kantuni kantuni deleted the remove/st-deck-gl-chart branch September 17, 2020 19:24
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.

Remove st.deck_gl_chart

4 participants