rusackas commented on code in PR #44130:
URL: https://github.com/apache/superset/pull/44130#discussion_r4080899942
##########
docs/docs/using-superset/using-ai-with-superset.mdx:
##########
@@ -267,6 +293,15 @@ Ask your admin for the MCP server URL and any
authentication tokens you need.
| `create_virtual_dataset` | Create a virtual dataset from a SQL query
|
| `update_dataset_metric` | Update a saved metric's expression, name,
verbose_name, or format (affects every chart using it; requires dataset
ownership) |
+### Semantic Layer
+
+| Tool | Description
|
+| ---------------------------- |
-----------------------------------------------------------------------------------------------------------------------------------
|
+| `list_metrics` | Discover metrics by name/description across
built-in datasets and external semantic views, with compatible dimensions
included |
+| `get_table` | Query a dataset or semantic view by metric
and dimension names, with optional filters, time range, and sorting
|
+| `get_compatible_dimensions` | Given metrics/dimensions already selected,
return the full set of dimensions compatible with that selection (may include
ones already selected) |
+| `get_compatible_metrics` | Given metrics/dimensions already selected,
return the metrics that can still be added (already-selected metrics excluded)
|
Review Comment:
@sadpandajoe Good catch, fixed the wording. Exclusion only holds for the
built-in path — external views just forward whatever the connector reports,
which can include already-selected metrics.
##########
docs/docs/using-superset/using-ai-with-superset.mdx:
##########
@@ -267,6 +293,15 @@ Ask your admin for the MCP server URL and any
authentication tokens you need.
| `create_virtual_dataset` | Create a virtual dataset from a SQL query
|
| `update_dataset_metric` | Update a saved metric's expression, name,
verbose_name, or format (affects every chart using it; requires dataset
ownership) |
+### Semantic Layer
+
+| Tool | Description
|
+| ---------------------------- |
-----------------------------------------------------------------------------------------------------------------------------------
|
+| `list_metrics` | Discover metrics by name/description across
built-in datasets and external semantic views, with compatible dimensions
included |
+| `get_table` | Query a dataset or semantic view by metric
and dimension names, with optional filters, time range, and sorting
|
+| `get_compatible_dimensions` | Given metrics/dimensions already selected,
return the full set of dimensions compatible with that selection (may include
ones already selected) |
+| `get_compatible_metrics` | Given metrics/dimensions already selected,
return the metrics that can still be added (already-selected metrics excluded)
|
Review Comment:
@sadpandajoe Good catch, fixed the wording. Exclusion only holds for the
built-in path. External views just forward whatever the connector reports,
which can include already-selected metrics.
--
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]
