search

microsoft / hi-ml

HI-ML toolbox for deep learning for medical imaging and Azure integration
https://aka.ms/hi-ml
MIT License
269 stars 60 forks source link

FIX: Fix empty list of script params not used #881

Closed fepegar closed 1 year ago

fepegar commented 1 year ago

Fix so no arguments are passed to the submitted script when an empty list is explicitly passed to script_params in submit_to_azure_if_needed.

codecov[bot] commented 1 year ago

Codecov Report

Merging #881 (64a2707) into main (7e8c8a0) will not change coverage. The diff coverage is 0.00%.

Impacted file tree graph

Flag Coverage Δ
hi-ml 83.01% <ø> (ø)
hi-ml-azure 25.38% <0.00%> (ø)
hi-ml-cpath 75.91% <ø> (ø)

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

Impacted Files Coverage Δ
hi-ml-azure/src/health_azure/himl.py 27.01% <0.00%> (ø)