sadpandajoe commented on code in PR #43771:
URL: https://github.com/apache/superset/pull/43771#discussion_r3989225934


##########
docs/docs/using-superset/using-ai-with-superset.mdx:
##########
@@ -341,3 +341,20 @@ Ask your admin for the MCP server URL and any 
authentication tokens you need.
 - Verify the connection in your AI client (e.g., the hammer icon in Claude 
Desktop)
 - Ask the AI "What Superset tools are available?" to confirm the connection
 - Restart your AI client if you recently changed the configuration
+
+
+### Sunburst chart data
+
+MCP chart tools support ordered Sunburst hierarchies with a primary metric and
+an optional secondary color metric. Use `chart_type: "sunburst"` in typed 
requests;

Review Comment:
   This wording suggests color encodes the secondary metric directly, but the 
implementation colors by the secondary-to-primary ratio, so a profit/revenue 
chart could be read as profit rather than margin. Could this say that the ratio 
drives the sequential color scale and that omitting it uses categorical colors?



-- 
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