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
f-string formatting #3277
f-string formatting #3277
Conversation
Hello @lhoestq, |
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.
Thanks a lot for all the changes ! It makes everything consistent and easier to read :)
I think I just spotted one issue:
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.
Alright it's all good now, thanks :)
Merging !
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.
At line 30 in benchmarks/format.py, shouldn't the new f-string also reference old_val
instead of new_val
?
Fix #3257
Replaced .format() and % by f-strings in the following modules :
Modules in src/Datasets/:
Module datasets will not be edited as asked by @mariosasko
-A correction of the first PR (#3267)-