Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Extending rotation_range to include roration_step #4755

Closed
guysoft opened this issue Dec 18, 2016 · 1 comment
Closed

Extending rotation_range to include roration_step #4755

guysoft opened this issue Dec 18, 2016 · 1 comment

Comments

@guysoft
Copy link

guysoft commented Dec 18, 2016 •
edited
Loading

Hey all,
This is my first PR attempt to Keras so I hope I am doing this right (aka following opening an issue before a PR).
I extended ImageDataGenerator to include a variable roration_step that lets you set discrete steps of angles here is the code.

I ran local tests here and it seems to work, but I didn't run the testing environment, I am not sure how to do that exactly.
Also this might be relevant to #3338

What else should I do to get this in? or get it to the right place to extending supports this?
Thanks!
@stale stale bot added the stale label May 23, 2017
@stale stale bot closed this as completed Jun 22, 2017
@guysoft
Copy link
Author

guysoft commented Jun 22, 2017

Well, it seems like no one is interested in explaining if this is PR worthy :-/

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@guysoft