[feiyun0112]

fix #7452

[Copilot]

Pull Request Overview

This PR addresses bug #7452 by fixing the incorrect handling of a non-default predicted label column name in the ImageClassificationTrainer scorer. Key changes include:

• Adding a new public field, PredictedLabelColumnName.
• Assigning the new field in the BindingsImpl constructor.
• Propagating the new predictedLabelColumnName parameter in ApplyToSchema.

Comments suppressed due to low confidence (1)

src/Microsoft.ML.Data/Scorers/PredictedLabelScorerBase.cs:118

• [nitpick] Consider adding tests to verify that a non-default 'PredictedLabelColumnName' is correctly handled throughout the binding and scoring processes.

return new BindingsImpl(input, rowMapper, Suffix, ScoreColumnKind, true, mapperScoreColumn, PredColType, PredictedLabelColumnName);

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.99%. Comparing base (f6221d5) to head (a82504b).
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7458      +/-   ##
==========================================
- Coverage   68.99%   68.99%   -0.01%     
==========================================
  Files        1482     1482              
  Lines      273879   273880       +1     
  Branches    28254    28254              
==========================================
- Hits       188965   188955      -10     
- Misses      77531    77544      +13     
+ Partials     7383     7381       -2     

Flag	Coverage Δ
Debug	68.99% <100.00%> (-0.01%)	⬇️
production	63.27% <100.00%> (-0.01%)	⬇️
test	89.46% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
...rosoft.ML.Data/Scorers/PredictedLabelScorerBase.cs	82.02% <100.00%> (+0.06%)	⬆️

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[michaelgsharp]

Looks good. Thanks for the fix!