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

Fixed calls to concat and convolution2d #1532

Merged

Conversation

andrewgilbert12
Copy link
Contributor

When attempting to run the adversarial crypto model, I got the following errors:

line 302, in _AssertCompatible
    (dtype.name, repr(mismatch), type(mismatch).__name__))
TypeError: Expected int32, got list containing Tensors of type '_Message' instead.
line 126, in model
    [tf.contrib.layers.fully_connected, tf.contrib.layers.convolution],
AttributeError: 'module' object has no attribute 'convolution'

It appears that the arguments for tf.concat were in the opposite order of the API documentation, and that the "tf.contrib.layers.convolution2d" method was called as "convolution". I have corrected both of these issues in this pull request.

@nealwu nealwu merged commit 2c4fea8 into tensorflow:master Jun 6, 2017
@nealwu
Copy link
Contributor

nealwu commented Jun 6, 2017

Thanks! cc @dave-andersen

@dave-andersen
Copy link
Contributor

Indeed - thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants