Staging to main: AzureML migration to v2 #2137
Merged
Conversation
This file contains 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
Signed-off-by: miguelgfierro <[email protected]>
Signed-off-by: miguelgfierro <[email protected]>
Signed-off-by: miguelgfierro <[email protected]>
Signed-off-by: miguelgfierro <[email protected]>
Signed-off-by: miguelgfierro <[email protected]>
Signed-off-by: miguelgfierro <[email protected]>
Signed-off-by: Simon Zhao <[email protected]>
Signed-off-by: Simon Zhao <[email protected]>
Signed-off-by: Simon Zhao <[email protected]>
Signed-off-by: miguelgfierro <[email protected]>
Signed-off-by: miguelgfierro <[email protected]>
Signed-off-by: miguelgfierro <[email protected]>
Signed-off-by: miguelgfierro <[email protected]>
Review test readme and define best practices
Signed-off-by: miguelgfierro <[email protected]>
Added free course on Recommendation systems
* Migrate AML SDK from v1 to v2 Signed-off-by: Simon Zhao <[email protected]> * Correct MLClient constructor paramenter names Signed-off-by: Simon Zhao <[email protected]> * Remove unsupported operation begin_start() on AmlCompute Signed-off-by: Simon Zhao <[email protected]> * Add label for environments.get() Signed-off-by: Simon Zhao <[email protected]> * Remove environment get Signed-off-by: Simon Zhao <[email protected]> * Update Signed-off-by: Simon Zhao <[email protected]> * Correct experiment and environment names Signed-off-by: Simon Zhao <[email protected]> * Correct compute Signed-off-by: Simon Zhao <[email protected]> * Create Conda env inside Dockerfile Signed-off-by: Simon Zhao <[email protected]> * Catch ResourceNotFoundError Signed-off-by: Simon Zhao <[email protected]> * Correct experiment name Signed-off-by: Simon Zhao <[email protected]> * Update env creation and job running Signed-off-by: Simon Zhao <[email protected]> * Try waiting for completion by stream Signed-off-by: Simon Zhao <[email protected]> * Try to fix conda activate Signed-off-by: Simon Zhao <[email protected]> * Import sys Signed-off-by: Simon Zhao <[email protected]> * Change logging level Signed-off-by: Simon Zhao <[email protected]> * Exit directly once pytest fails Signed-off-by: Simon Zhao <[email protected]> * Set numpy<2.0.0 due to issue of cornac Signed-off-by: Simon Zhao <[email protected]> * Correct Dockerfile Signed-off-by: Simon Zhao <[email protected]> * Change heredoc Signed-off-by: Simon Zhao <[email protected]> * Set dockerfile version Signed-off-by: Simon Zhao <[email protected]> * Copy environment.yml to container Signed-off-by: Simon Zhao <[email protected]> * Update .github/actions/azureml-test/action.yml Co-authored-by: Miguel Fierro <[email protected]> * Update as suggested by Andreas * Print pytest logs Signed-off-by: Simon Zhao <[email protected]> * Group outputs Signed-off-by: Simon Zhao <[email protected]> * Install pip in conda env, and show pytest warnings Signed-off-by: Simon Zhao <[email protected]> * Add command name Signed-off-by: Simon Zhao <[email protected]> * Update Signed-off-by: Simon Zhao <[email protected]> * Show warnings in pytest Signed-off-by: Simon Zhao <[email protected]> * Show warnings in pytest Signed-off-by: Simon Zhao <[email protected]> --------- Signed-off-by: Simon Zhao <[email protected]> Co-authored-by: Miguel Fierro <[email protected]>
miguelgfierro
requested review from
gramhagen,
anargyri,
loomlike,
wutaomsft and
SimonYansenZhao
as code owners
SimonYansenZhao
approved these changes
Jul 31, 2024
github-actions bot
pushed a commit
that referenced
this pull request
Jul 31, 2024
Staging to main: AzureML migration to v2
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
Adding the latest changes in AzureML.
Related Issues
There is still this issue unfixed #2133 which is breaking the tests (both stating and main). However, it is not a problem with the code itself, it is just that the dataset URL is broken.
References
Checklist:
git commit -s -m "your commit message".staging branchAND NOT TOmain branch.