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 issue in K.logsumexp in tensorflow backend #6516

Merged
merged 1 commit into from
May 5, 2017

Conversation

linxihui
Copy link
Contributor

@linxihui linxihui commented May 5, 2017

Argument reduction_indices is depreciated in tensorflow. It also causes shape inference issue for placeholder, example

x = K.placeholder((None, 9, 9))

y1 = tf.reduce_logsumexp(x, reduction_indices=2)
K.int_shape(y1)  # gives None

y2 = tf.reduce_logsumexp(x, axis=2)
K.int_shape(y2)  # gives (None, 9)

@fchollet
Copy link
Collaborator

fchollet commented May 5, 2017

Correct, thank you.

@fchollet fchollet merged commit 61c9cdc into keras-team:master May 5, 2017
@gitfourteen
Copy link

What is the proper way to use K.logsumexp for now? Seems like there is no such things

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

Successfully merging this pull request may close these issues.

3 participants