aminghadersohi commented on code in PR #43771: URL: https://github.com/apache/superset/pull/43771#discussion_r3989729693
########## 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: Addressed in fe5d57b096. The public Sunburst docs and schema now state that the secondary-to-primary ratio drives the sequential color scale (for example, profit/revenue represents margin), while omitting `secondary_metric` uses categorical arc colors. I aligned the MCP capability text, generated schema description, plugin validation detail, chart-config guidance, and `generate_chart` tool description, and added `test_secondary_metric_schema_explains_ratio_color_semantics` to lock the ratio/sequential/categorical wording. Focused Sunburst + Gauge tests pass: 458 passed, 1 skipped; staged-file pre-commit also passes. -- 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]
