-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AzureML tests: Durations, disable warnings, logs less verbose #1864
Conversation
Error:
fixed adding the argument as a string:
|
Error with
Tried [DIDN'T WORK]:
Tried:
however, I still see the warning: https://github.com/microsoft/recommenders/actions/runs/4064811280/jobs/6998690821 Also, right now the tests are broken due to #1876 action.yml executes In submit_groupwise_azureml_pytest.py it is called FIXED in a9baaa8 |
Error in
I get:
The issue came because I modified the deprecated code:
this saves the current branch to a variable, that can be later obtained via Tried:
|
@simonzhaoms @pradnyeshjoshi this PR is ready, it adds durations, disable warnings and the output is less verbose. However, there is an error with AzureML #1876. We can either put this into staging, and then fix #1876 or we can wait to fix #1876, update this branch and then merge it. What do you think? |
After merging #1879, I updated this branch |
Description
Add parameters to pytest: durations and disable warnings
Related Issues
Fix #1857
References
Checklist:
staging branch
and not tomain branch
.