codeant-ai-for-open-source[bot] commented on code in PR #43571:
URL: https://github.com/apache/superset/pull/43571#discussion_r3961078015
##########
superset/mcp_service/chart/schemas.py:
##########
@@ -1045,6 +1045,57 @@ def reject_sql_expression_on_dimensions(self) ->
"PieChartConfig":
return self
+class RadarChartConfig(BaseChartConfig):
+ """Config for radar charts (viz_type ``radar``).
+
+ Matches the frontend Radar buildQuery contract: multiple ``metrics``
+ become the radar axes (indicators), and an optional ``groupby`` splits the
+ data into one polygon per category. With no groupby the whole dataset is a
+ single polygon. The query orders by the first metric descending.
+ """
+
+ model_config = ConfigDict(extra="ignore", populate_by_name=True)
+
+ chart_type: Literal["radar"] = "radar"
+ metrics: List[ColumnRef] = Field(
+ ...,
+ min_length=1,
Review Comment:
✅ **Customized review instruction saved!**
**Instruction:**
> For radar chart validation, allow at least one metric and keep the schema,
pre-validation message, and schema error hint consistent; describe multiple
metrics as a readability recommendation rather than a requirement.
**Applied to:**
- `superset/mcp_service/chart/**`
---
💡 *To manage or update this instruction, visit: [CodeAnt AI
Settings](https://app.codeant.ai/org/settings/learnings)*
--
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]
