bito-code-review[bot] commented on PR #35844:
URL: https://github.com/apache/superset/pull/35844#issuecomment-3446883172
<details open>
<summary><strong>Interaction Diagram by <a
href="https://bito.ai#sequence_diagram";>Bito</a></strong></summary>
```mermaid
sequenceDiagram
participant API as REST API Layer
participant QC as ChartDataQueryContextSchema
participant PO as ChartDataProphetOptionsSchema<br/>🔄 Updated | ●●○ Medium
participant Validator as Range Validator<br/>🟩 Added | ●●○ Medium
participant Prophet as prophet() Function
Note over PO: Marshmallow 4.0+ compatibility<br/>Replace min param with
validate=Range()
API->>QC: POST /api/v1/chart/data
QC->>PO: Load post_processing options
PO->>Validator: Validate periods field
Validator-->>PO: periods >= 0 check
PO-->>Prophet: Pass validated options
Prophet-->>API: Return forecast results
```
Critical path: REST API
Layer->ChartDataQueryContextSchema->ChartDataProphetOptionsSchema->Range
Validator->prophet() Function
> **Note:** The periods field in ChartDataProphetOptionsSchema is updated to
use Marshmallow 4.0+ compatible Range validator instead of deprecated min
parameter. This ensures compatibility with newer Marshmallow versions while
maintaining the same validation logic (periods must be >= 0).
</details>
--
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]
