feat(automl): add Prediction type column to AutoML runs table#7260
Conversation
Display the task_type for each run as a friendly label (e.g., "Binary classification") after the Description column. Extract shared getTaskType utility to consolidate fallback logic across AutomlRunsTableRow and AutomlResultsContext. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Skipping CI for Draft Pull Request. |
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 47 minutes and 9 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: GAUNSD The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
cae3a09
into
opendatahub-io:main
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7260 +/- ##
==========================================
+ Coverage 64.80% 64.82% +0.01%
==========================================
Files 2441 2442 +1
Lines 75996 76015 +19
Branches 19158 19164 +6
==========================================
+ Hits 49253 49276 +23
+ Misses 26743 26739 -4 see 9 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
https://issues.redhat.com/browse/RHOAIENG-58441
Description
Adds a Prediction type column to the AutoML runs table, displayed after the Description column. The column shows the friendly label for the run's
task_type(e.g., "Binary classification", "Regression", "Time series forecasting") using the existingTASK_TYPE_LABELSmapping.Changes:
columns.ts— Added a"Prediction type"column and rebalanced column widths (20/25/15/15/10) with Description remaining the largest.AutomlRunsTableRow.tsx— Renders the prediction type label using the newgetTaskTypeutility. Shows—whenruntime_config.parametersis absent; defaults to "Time series forecasting" when parameters exist buttask_typeis missing.utils.ts— Extracted a sharedgetTaskType(pipelineRun)utility that encapsulates the task type extraction and fallback logic. Also refactoredisTabularRunto use it internally.AutomlResultsContext.ts— Refactored to usegetTaskTypeinstead of inlinehasOwnPropertychecks, consolidating the fallback logic in one place.How Has This Been Tested?
AutomlRunsTableRow.spec.tsx— all 25 tests pass.AutomlResultsContext.spec.tsxtests pass (19 tests).Test Impact
Added 3 new unit tests in
AutomlRunsTableRow.spec.tsx:should render friendly prediction type label— verifies'binary'renders as"Binary classification"should render em dash when runtime_config is missing— verifies—whenruntime_configis undefinedshould default prediction type to time series forecasting when task_type is missing— verifies the timeseries fallback when parameters exist buttask_typeis absentRequest review criteria:
Self checklist (all need to be checked):
If you have UI changes:
After the PR is posted & before it merges:
main