Add Kumo-Forecast to SDM’s Kumo time-series models. - #1048
agautam478 wants to merge 1 commit into
Conversation
agautam478
commented
Oct 5, 2026
Signed-off-by: Aditi Gautam <adgautam@nvidia.com>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. 🧰 Additional context used📚 Code guidelines (1)No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (22)
Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review. 📝 SummarySummary by CodeRabbit
WalkthroughAdds ChangesKumo forecasting
Priority: ➖ Normal Estimated code review effort: 4 (Complex) | ~45 minutes Sequence Diagram(s)sequenceDiagram
participant Caller
participant KumoForecasting
participant RevIN
participant PatchEmbedding
participant T5Encoder
participant CrossChannelAttention
participant ForecastingHead
Caller->>KumoForecasting: Submit context and forecast horizon
KumoForecasting->>RevIN: Normalize time-series inputs
KumoForecasting->>PatchEmbedding: Create patch embeddings
KumoForecasting->>T5Encoder: Encode patch embeddings
KumoForecasting->>CrossChannelAttention: Mix variates when enabled
KumoForecasting->>ForecastingHead: Project encoded patches to forecast
KumoForecasting->>RevIN: Restore forecast scale
KumoForecasting-->>Caller: Return forecast and target columns
Merge Risk: ⚪ Minimal · up to This adds a new forecasting model with tests, docs and an example. No concrete merge-blocking risk was identified in the supplied changes. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 23.53% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 68 functions across 20 files. (2 skipped: 2 unsupported.)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |