aminghadersohi commented on code in PR #37639:
URL: https://github.com/apache/superset/pull/37639#discussion_r2760179157
##########
superset/mcp_service/chart/schemas.py:
##########
@@ -692,7 +696,11 @@ class XYChartConfig(BaseModel):
False,
description="Stack bars/areas on top of each other instead of
side-by-side",
)
- group_by: ColumnRef | None = Field(None, description="Column to group by")
+ group_by: ColumnRef | None = Field(
+ None,
+ description="Column to group by (creates series/breakdown). "
+ "Use this field for series grouping — do NOT use 'series'.",
Review Comment:
Good to know, thanks for the heads up. We can revisit this when we add
support for those viz types — at that point we'd likely need separate config
schemas for bubble/word cloud anyway since they have different field
requirements.
--
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]
