dosubot[bot] commented on issue #36838:
URL: https://github.com/apache/superset/issues/36838#issuecomment-3691415878

   <!-- Greeting -->
   Hi @ekatrif! I'm [Dosu](https://go.dosu.dev/dosubot) and I’m helping the 
superset team.
   
   <!-- Answer -->
   This error happens because setting a very small cell size in the deck.gl 
Contour chart causes the number of grid cells (and thus the data buffer sent to 
WebGL) to explode, exceeding the 32-bit buffer size limit and freezing the UI. 
Superset passes your cell size value directly to deck.gl without any validation 
or upper/lower bounds, so extreme values can easily trigger this issue [see 
code 
reference](https://github.com/apache/superset/blob/829e4d92d91ceae4b43b1ed3b063ffe45377799c/superset-frontend/plugins/legacy-preset-chart-deckgl/src/layers/Contour/Contour.tsx#L19-L97).
   
   To avoid this, use a larger cell size (e.g., 200 or 300 meters) and avoid 
very small values. There’s currently no built-in safeguard in Superset to 
prevent this, so you’ll need to choose values that keep the grid resolution 
reasonable for your dataset and viewport.
   
   If you want to prevent this in the UI, it would require adding validation or 
min/max limits to the cell size control. Right now, Superset acts as a thin 
wrapper over deck.gl and does not handle these errors, so the best workaround 
is to avoid small cell sizes that could cause buffer overflows.
   
   
   <!-- Dosu Comment Footer -->
   *To reply, just mention [@dosu](https://go.dosu.dev/dosubot).*
   
   ---
   <sup>How did I do? 
[Good](https://app.dosu.dev/response-feedback/beef718a-8462-4d61-a175-fd4dbb85a783?feedback_type=great_response)
 | 
[Irrelevant](https://app.dosu.dev/response-feedback/beef718a-8462-4d61-a175-fd4dbb85a783?feedback_type=irrelevant_answer)
 | 
[Incorrect](https://app.dosu.dev/response-feedback/beef718a-8462-4d61-a175-fd4dbb85a783?feedback_type=incorrect_sources)
 | 
[Verbose](https://app.dosu.dev/response-feedback/beef718a-8462-4d61-a175-fd4dbb85a783?feedback_type=too_verbose)
 | 
[Hallucination](https://app.dosu.dev/response-feedback/beef718a-8462-4d61-a175-fd4dbb85a783?feedback_type=hallucination)
 | [Report 
🐛](https://app.dosu.dev/response-feedback/beef718a-8462-4d61-a175-fd4dbb85a783?feedback_type=bug_report)
 | 
[Other](https://app.dosu.dev/response-feedback/beef718a-8462-4d61-a175-fd4dbb85a783?feedback_type=other)</sup>
   
   [![Chat with 
Dosu](https://dosu.dev/dosu-chat-badge.svg)](https://app.dosu.dev/a28d3c7e-a9d3-459e-9fb6-3a6f9ff4f357/ask?utm_source=github)&nbsp;[![Open
 in 
Cursor](https://dosu.dev/dosu-in-cursor.svg)](https://cursor.com/link/prompt?text=This%20error%20happens%20because%20setting%20a%20very%20small%20cell%20size%20in%20the%20deck.gl%20Contour%20chart%20causes%20the%20number%20of%20grid%20cells%20%28and%20thus%20the%20data%20buffer%20sent%20to%20WebGL%29%20to%20explode%2C%20exceeding%20the%2032-bit%20buffer%20size%20limit%20and%20freezing%20the%20UI.%20Superset%20passes%20your%20cell%20size%20value%20directly%20to%20deck.gl%20without%20any%20validation%20or%20upper/lower%20bounds%2C%20so%20extreme%20values%20can%20easily%20trigger%20this%20issue%C2%A0%5Bsee%20code%20reference%5D%28https%3A//github.com/apache/superset/blob/829e4d92d91ceae4b43b1ed3b063ffe45377799c/superset-frontend/plugins/legacy-preset-chart-deckgl/src/layers/Contour/Contour.tsx%23L19-L97%29.%0A%0ATo%20avoid%20this%2
 
C%20use%20a%20larger%20cell%20size%20%28e.g.%2C%20200%20or%20300%20meters%29%20and%20avoid%20very%20small%20values.%20There%E2%80%99s%20currently%20no%20built-in%20safeguard%20in%20Superset%20to%20prevent%20this%2C%20so%20you%E2%80%99ll%20need%20to%20choose%20values%20that%20keep%20the%20grid%20resolution%20reasonable%20for%20your%20dataset%20and%20viewport.%0A%0AIf%20you%20want%20to%20prevent%20this%20in%20the%20UI%2C%20it%20would%20require%20adding%20validation%20or%20min/max%20limits%20to%20the%20cell%20size%20control.%20Right%20now%2C%20Superset%20acts%20as%20a%20thin%20wrapper%20over%20deck.gl%20and%20does%20not%20handle%20these%20errors%2C%20so%20the%20best%20workaround%20is%20to%20avoid%20small%20cell%20sizes%20that%20could%20cause%20buffer%20overflows.)&nbsp;[![Join
 
Discord](https://img.shields.io/badge/join-5865F2?logo=discord&logoColor=white&label=)](https://go.dosu.dev/discord-bot)&nbsp;[![Share
 on X](https://img.shields.io/badge/X-share-black)](https://twitter.com/intent
 
/tweet?text=%40dosu_ai%20helped%20me%20solve%20this%20issue!&url=https%3A//github.com/apache/superset/issues/36838)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to