Skip to content
This repository was archived by the owner on Nov 21, 2025. It is now read-only.

Conversation

@ant0nsc
Copy link
Collaborator

@ant0nsc ant0nsc commented Jul 25, 2023

When submitting a script with a path relative to current working directory, SDK v2 raises an error, even though the script exists.

Closes #890

@codecov
Copy link

codecov bot commented Jul 25, 2023

Codecov Report

Merging #904 (2d678df) into main (1a754d8) will decrease coverage by 0.14%.
The diff coverage is n/a.

Impacted file tree graph

Flag Coverage Δ
hi-ml 82.41% <ø> (-0.89%) ⬇️
hi-ml-cpath 76.17% <ø> (ø)

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

Files Coverage Δ
hi-ml-azure/src/health_azure/himl.py 43.75% <ø> (+1.76%) ⬆️
hi-ml-azure/src/health_azure/utils.py 29.39% <ø> (ø)

... and 2 files with indirect coverage changes

Copy link
Contributor

@kenza-bouzid kenza-bouzid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@ant0nsc ant0nsc merged commit f352519 into main Nov 2, 2023
@ant0nsc ant0nsc deleted the antonsc/path_resolve branch November 2, 2023 14:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Job submission of scripts in subfolders of cwd fails, saying the script should be under project root

4 participants