-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Conversation
Hey @hankaj , Thanks for submitting the PR
CI supported jobs: [clang, miscellaneous, windows-gpu, edge, unix-cpu, centos-gpu, centos-cpu, website, windows-cpu, unix-gpu, sanity] Note: |
@mxnet-bot run ci [unix-gpu] |
Jenkins CI successfully triggered : [unix-gpu] |
'Please train the models and run inference again' % bucket_name) | ||
raise Exception("No trained models found in S3 bucket : %s for this file. " | ||
"Please train the models and run inference again" % bucket_name) | ||
logging.error('No trained models found in S3 bucket : {} for this file. ' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the reason to have to different approaches than in line 110?
here {bucket_name} seems to be more readable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In CI I got the error that you should not use f-strings in messages of logging functions.
logging.error('No trained models found in S3 bucket : {} for this file. ' | ||
'Please train the models and run inference again'.format(bucket_name)) | ||
raise Exception(f"No trained models found in S3 bucket : {bucket_name} for this file. " | ||
"Please train the models and run inference again") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same comment as above
logging.error('No trained models found in S3 bucket : {} for this file. ' | |
'Please train the models and run inference again'.format(bucket_name)) | |
raise Exception(f"No trained models found in S3 bucket : {bucket_name} for this file. " | |
"Please train the models and run inference again") | |
logging.error(f'No trained models found in S3 bucket : {bucket_name} for this file. ' | |
'Please train the models and run inference again') | |
raise Exception(f"No trained models found in S3 bucket : {bucket_name} for this file. " | |
"Please train the models and run inference again") |
@mxnet-bot run ci [sanity] |
Jenkins CI successfully triggered : [sanity] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reviewed 2x - LGTM
@mxnet-bot run ci [all] |
Jenkins CI successfully triggered : [sanity, centos-cpu, windows-cpu, miscellaneous, windows-gpu, unix-cpu, clang, unix-gpu, centos-gpu, website, edge] |
@mxnet-bot run ci [unix-gpu] |
Jenkins CI successfully triggered : [unix-gpu] |
This PR changes most of the
%
-formatting in python to f-strings (orformat()
in some cases).According to Python documentation f-strings and
format()
are preferred to the%
-formatting (which was used in Python 2).