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
Axes ordering handling and unit tests for GaussianVisLayer in models.dbm.layer #681
Conversation
Retrieving recent pylearn2 additions before finalizing dbm unit tests
this merge is necessary,
if nvis is None: | ||
assert rows is not None | ||
assert cols is not None | ||
assert channels is not None | ||
self.space = Conv2DSpace(shape=[rows,cols], num_channels=channels, axes=axes) | ||
# To make GaussianVisLayer compatible with any axis ordering |
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.
I won't make you fix this to get the PR merged, but according to PEP8 style there should be a space on both sides of the = sign. (I know most of the library is not PEP8 compliant, but we're trying to move toward it being so)
Looks good overall. There are two small formatting problems, but I won't enforce these at the moment, just read my comments so you'll know for next time. Pretty soon we'll have tests that automatically enforce the format requirements. |
Axes ordering handling and unit tests for GaussianVisLayer in models.dbm.layer
-Addressed NotImplementedErrors in dbm.layer.GaussianVisLayer regarding axes ordering for Conv2DSpace shape
-Added basic unit tests for GaussianVisLayer