microsoft / computervision-recipes

Best Practices, code samples, and documentation for Computer Vision.
MIT License
9.55k stars 1.18k forks source link

Missing `f` prefix on f-strings #668

Closed code-review-doctor closed 2 years ago

code-review-doctor commented 2 years ago

Some strings looks like they're meant to be f-strings but are missing the f prefix meaning variable interpolation won't happen.

https://github.com/microsoft/computervision-recipes/blob/9380789c5fe47069aa58b33b59fdb15ead528e84/utils_cv/classification/parameter_sweeper.py#L152 https://github.com/microsoft/computervision-recipes/blob/9380789c5fe47069aa58b33b59fdb15ead528e84/utils_cv/classification/parameter_sweeper.py#L156 https://github.com/microsoft/computervision-recipes/blob/9380789c5fe47069aa58b33b59fdb15ead528e84/utils_cv/classification/data.py#L99

I found this issue automatically. I'm a bot. Beep Boop 🦊. See other issues I found in your repo here