Skip to content
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

The values of degrees parameter of RandomRotation() are inclusive so they should be [min, max] and [-degrees, +degrees] #8831

Open
hyperkai opened this issue Dec 26, 2024 · 0 comments

Comments

@hyperkai
Copy link

📚 The doc issue

The doc of RandomRotation() says that the range of degrees parameter is (min, max) and (-degrees, +degrees) as shown below but actually, the values of degrees parameter are inclusive:

Parameters:

  • degrees (sequence or number) – Range of degrees to select from. If degrees is a number instead of sequence like (min, max), the range of degrees will be (-degrees, +degrees).

Suggest a potential alternative/fix

So they should be [min, max] and [-degrees, +degrees] as shown below:

Parameters:

  • degrees (sequence or number) – Range of degrees to select from. If degrees is a number instead of sequence like [min, max], the range of degrees will be [-degrees, +degrees].
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant