Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Fix softmax_cross_entropy list input names #6766

Closed
wants to merge 5 commits into from
Closed

Fix softmax_cross_entropy list input names #6766

wants to merge 5 commits into from

Conversation

kevinthesun
Copy link
Contributor

  1. Add FListInputNames attribute to softmax_cross_entropy operator.
  2. Add softmax_cross_entropy unit test.

@kevinthesun kevinthesun mentioned this pull request Jun 29, 2017
Open
@geoalgo
Copy link
Contributor

geoalgo commented Jul 10, 2017

Thanks for fixing this, I am also impacted by this problem and look forward for your patch.

@wielandbrendel
Copy link

Thanks for the fix @kevinthesun! What is holding this PR back?

@kevinthesun
Copy link
Contributor Author

@piiswrong Do we still need this fix?

@wielandbrendel
Copy link

I came across this error when compiling the current master, and so I needed to manually pull this PR locally to fix this problem.

@eric-haibin-lin
Copy link
Member

Moved to #8254

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@kevinthesun @geoalgo @wielandbrendel @eric-haibin-lin