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

deck.gl Multiple Layers chart broke after 1.2.0 #16572

Closed
zgramana opened this issue Sep 2, 2021 · 4 comments
Closed

deck.gl Multiple Layers chart broke after 1.2.0 #16572

zgramana opened this issue Sep 2, 2021 · 4 comments
Assignees
Labels
#bug Bug report viz:charts:deck.gl Related to deck.gl charts

Comments

@zgramana
Copy link

zgramana commented Sep 2, 2021

deck.gl Multiple Layers chart is broken, causing the Explore page to fail to render

Expected results

Open a deck.gl Multiple Layer chart see a map rendered

Actual results

The entire page below the menu bar disappears as soon as the "Run Query" button is opened

Screenshots

image

How to reproduce the bug

  1. Create a deck.gl Multiple Layer chart,
  2. Add "Path" chart as the layer
  3. Press "Run Query"

Environment

Currently using via docker-compose -f docker-compose-non-dev.yml up

  • superset version: DockerHub apache/superset@sha256:c6956897004b8072f97ee8e1a9ea245284b373712c96929ab13ca954164e2348
  • python version: Python 3.7.9
  • node.js version: node -v
  • any feature flags active:
FEATURE_FLAGS = {
    "ALERT_REPORTS": True,
    "DASHBOARD_NATIVE_FILTERS": True,
    "DASHBOARD_CROSS_FILTERS": True,
    "DASHBOARD_NATIVE_FILTERS_SET": True,
    "DASHBOARD_FILTERS_EXPERIMENTAL": True,
    "OMNIBAR" : True,
    "TAGGING_SYSTEM" : True,
    "DYNAMIC_PLUGINS" : True,
}

Checklist

Make sure to follow these steps before submitting your issue - thank you!

  • [x ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
  • [x ] I have reproduced the issue with at least the latest released version of superset.
  • [x ] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context

Observed errors in the JS console:

image

@zgramana zgramana added the #bug Bug report label Sep 2, 2021
@rosemarie-chiu
Copy link
Contributor

@junlincc @evans this issue is reported again today

@junlincc
Copy link
Member

junlincc commented Oct 25, 2021

Prioritizing! @zhaoyongjie please help @stephenLYZ work on this after moving deck.gl repo to the main repo.
Let's get it fixed ASAP

@junlincc junlincc added the viz:charts:deck.gl Related to deck.gl charts label Oct 25, 2021
@zhaoyongjie
Copy link
Member

zhaoyongjie commented Oct 28, 2021

There is an open PR for this. cc: @junlincc @rosemarie-chiu @stephenLYZ
apache-superset/superset-ui#1376

@rusackas
Copy link
Member

Resolved by #17703 - let us know if there are more problems remaining that mean we should reopen this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
#bug Bug report viz:charts:deck.gl Related to deck.gl charts
Projects
None yet
Development

No branches or pull requests

5 participants