i don't understand why this got so many "best practices" votes

In this case is ok, since the parameters are integers and float numbers, so changing them inside the function will not change the value in the caller. If instead this was a list, a set, or other types that are passed as refrences then changing the value inside the function could cause side effects outside the function