-
Notifications
You must be signed in to change notification settings - Fork 3
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
Celeba and Label-dependent transformations. #236
Conversation
Tests pass! |
tests/vision_test.py
Outdated
greyscale=False, | ||
) | ||
|
||
augmented_data = colorizer(data, labels) |
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.
are there any assertions that you can make about the augmented data?
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.
Such assertions would be easy enough to devise despite my earlier grounsing about it. To check its basic functionality, for instance, simply apply the transform with scale 0 to matrices of zeroes, one for each colour and check for correspondence with the predefined mean rgb values.
Should we get this merged in? |
If it's up to your standards, absolutely. |
@thomkeh ? |
Yeah, maybe put a to-do in the test. |
Co-Authored-By: thomkeh <7741417+thomkeh@users.noreply.github.com>
Sorry, that to-do comment wasn't properly indented. My fault! |
It wasn't even a comment! |
No description provided.