timeseries: Welding - Added training scripts for classification with scikit learn#2354
Open
sathyendranv wants to merge 26 commits intoopen-edge-platform:mainfrom
Open
Conversation
…scikit learn Signed-off-by: Vellaisamy, Sathyendran <sathyendran.vellaisamy@intel.com>
Signed-off-by: Vellaisamy, Sathyendran <sathyendran.vellaisamy@intel.com>
Signed-off-by: Vellaisamy, Sathyendran <sathyendran.vellaisamy@intel.com>
Signed-off-by: Vellaisamy, Sathyendran <sathyendran.vellaisamy@intel.com>
Signed-off-by: Sathyendran Vellaisamy <sathyendran.vellaisamy@intel.com>
Signed-off-by: Vellaisamy, Sathyendran <sathyendran.vellaisamy@intel.com>
Signed-off-by: Vellaisamy, Sathyendran <sathyendran.vellaisamy@intel.com>
Signed-off-by: Vellaisamy, Sathyendran <sathyendran.vellaisamy@intel.com>
vkb1
reviewed
Apr 6, 2026
...s-time-series/apps/weld-anomaly-detection/time-series-analytics-config/udfs/requirements.txt
Show resolved
Hide resolved
...rial-edge-insights-time-series/apps/weld-anomaly-detection/classification-training/README.md
Outdated
Show resolved
Hide resolved
...rial-edge-insights-time-series/apps/weld-anomaly-detection/classification-training/README.md
Show resolved
Hide resolved
...rial-edge-insights-time-series/apps/weld-anomaly-detection/classification-training/README.md
Outdated
Show resolved
Hide resolved
...ge-insights-time-series/apps/weld-anomaly-detection/classification-training/requirements.txt
Outdated
Show resolved
Hide resolved
manufacturing-ai-suite/industrial-edge-insights-time-series/docker-compose.yml
Show resolved
Hide resolved
Signed-off-by: Vellaisamy, Sathyendran <sathyendran.vellaisamy@intel.com>
Signed-off-by: Vellaisamy, Sathyendran <sathyendran.vellaisamy@intel.com>
Signed-off-by: Vellaisamy, Sathyendran <sathyendran.vellaisamy@intel.com>
Signed-off-by: Vellaisamy, Sathyendran <sathyendran.vellaisamy@intel.com>
Signed-off-by: Vellaisamy, Sathyendran <sathyendran.vellaisamy@intel.com>
vkb1
approved these changes
Apr 10, 2026
...e-series/apps/weld-anomaly-detection/classification-training/weld_defect_inference_sample.py
Show resolved
Hide resolved
Signed-off-by: Vellaisamy, Sathyendran <sathyendran.vellaisamy@intel.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the weld anomaly detection sample to use a scikit-learn (Intel-accelerated) classifier pipeline for inference, and adds companion training/inference scripts to generate and validate the exported model artifacts.
Changes:
- Replace CatBoost-based weld UDF inference with a joblib-loaded scikit-learn pipeline + optional explanation payload.
- Add a classification training package (training script, predictor/CLI, sample inference scripts, README, and Python requirements).
- Tighten PR workflow path filters and add log-based readiness checks after sample-app deployment.
Reviewed changes
Copilot reviewed 12 out of 16 changed files in this pull request and generated 11 comments.
Show a summary per file
| File | Description |
|---|---|
| manufacturing-ai-suite/industrial-edge-insights-time-series/apps/weld-anomaly-detection/time-series-analytics-config/udfs/weld_anomaly_detector.py | Switch UDF inference to sklearn pipeline, add explanation + Intel offload config usage. |
| manufacturing-ai-suite/industrial-edge-insights-time-series/apps/weld-anomaly-detection/time-series-analytics-config/tick_scripts/weld_anomaly_detector.tick | Minor formatting change. |
| manufacturing-ai-suite/industrial-edge-insights-time-series/apps/weld-anomaly-detection/time-series-analytics-config/models/weld_anomaly_detector.pkl | Add exported sklearn pipeline artifact for UDF inference. |
| manufacturing-ai-suite/industrial-edge-insights-time-series/apps/weld-anomaly-detection/time-series-analytics-config/models/weld_anomaly_detector_labels.pkl | Add exported LabelEncoder artifact for decoding class indices. |
| manufacturing-ai-suite/industrial-edge-insights-time-series/apps/weld-anomaly-detection/time-series-analytics-config/models/weld_anomaly_detector.json | Add model metadata + per-class feature stats used for explanations. |
| manufacturing-ai-suite/industrial-edge-insights-time-series/apps/weld-anomaly-detection/time-series-analytics-config/models/weld_anomaly_detector.txt | Add evaluation report for the exported classifier. |
| manufacturing-ai-suite/industrial-edge-insights-time-series/apps/weld-anomaly-detection/time-series-analytics-config/config.json | Point config to the new .pkl model and set device selection. |
| manufacturing-ai-suite/industrial-edge-insights-time-series/apps/weld-anomaly-detection/classification-training/weld_defect_train.py | New training script to build/export the sklearn model artifacts. |
| manufacturing-ai-suite/industrial-edge-insights-time-series/apps/weld-anomaly-detection/classification-training/weld_defect_predict.py | New predictor class + CLI for row-by-row inference. |
| manufacturing-ai-suite/industrial-edge-insights-time-series/apps/weld-anomaly-detection/classification-training/weld_defect_inference_sample.py | New sample script for CSV/demo inference (no explanation). |
| manufacturing-ai-suite/industrial-edge-insights-time-series/apps/weld-anomaly-detection/classification-training/weld_defect_inference_sample_w_explanation.py | New sample script for CSV/demo inference including explanation block. |
| manufacturing-ai-suite/industrial-edge-insights-time-series/apps/weld-anomaly-detection/classification-training/requirements.txt | New pinned Python dependencies for training/inference scripts. |
| manufacturing-ai-suite/industrial-edge-insights-time-series/apps/weld-anomaly-detection/classification-training/README.md | New end-to-end documentation for training + inference usage. |
| .github/workflows/industrial-edge-insights-time-series-pull-request.yml | Broaden path globs and add “wait for Kapacitor tasks enabled” log checks. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
...eries/apps/weld-anomaly-detection/time-series-analytics-config/udfs/weld_anomaly_detector.py
Show resolved
Hide resolved
...eries/apps/weld-anomaly-detection/time-series-analytics-config/udfs/weld_anomaly_detector.py
Outdated
Show resolved
Hide resolved
...eries/apps/weld-anomaly-detection/time-series-analytics-config/udfs/weld_anomaly_detector.py
Outdated
Show resolved
Hide resolved
...eries/apps/weld-anomaly-detection/time-series-analytics-config/udfs/weld_anomaly_detector.py
Outdated
Show resolved
Hide resolved
...nsights-time-series/apps/weld-anomaly-detection/classification-training/weld_defect_train.py
Outdated
Show resolved
Hide resolved
...e-series/apps/weld-anomaly-detection/classification-training/weld_defect_inference_sample.py
Show resolved
Hide resolved
...eries/apps/weld-anomaly-detection/time-series-analytics-config/udfs/weld_anomaly_detector.py
Show resolved
Hide resolved
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
timeseries: Welding - Added training scripts for classification with scikit learn
Fixes # (issue)
Any Newly Introduced Dependencies
NA
How Has This Been Tested?
NA
Checklist: