Optimized the standardization process by combining the two loops and improving efficiency.
Added return statement where we intended to return but didn't implement
Motivation and Context
Removed the redundant loop that iterated over standardize_result to update ready.id. Instead, ready.id is now directly assigned the values from the current iteration's standardize_result if is_standardized is TRUE every time we perform a standardization.
For now, I've commented out the previous changes and will remove them once the current changes are approved.
Review Time Estimate
[ ] Immediately
[ ] Within one week
[x] When possible
Types of changes
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)
Checklist:
[ ] My change requires a change to the documentation.
Description
return
statement where we intended to return but didn't implementMotivation and Context
Removed the redundant loop that iterated over
standardize_result
to updateready.id
. Instead,ready.id
is now directly assigned the values from the current iteration'sstandardize_result
ifis_standardized
is TRUE every time we perform a standardization.For now, I've commented out the previous changes and will remove them once the current changes are approved.
Review Time Estimate
Types of changes
Checklist: