Skip to content

Fix bug in SequenceSoftmax #3058

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

Merged

Conversation

reyoung
Copy link
Collaborator

@reyoung reyoung commented Jul 25, 2017

Also remove operator bool in Error. The Error should be removed
later because it is not necessary for Paddle. We are now using Enforce
to handle error.

@reyoung reyoung requested a review from pengli09 July 25, 2017 10:33
Copy link
Contributor

@pengli09 pengli09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@reyoung reyoung force-pushed the feature/remove_error_in_paddle branch from c831d40 to 5e9f67b Compare July 25, 2017 11:25
Also remove operator bool in Error. The Error should be removed
later because it is not necessary for Paddle. We are now using Enforce
to handle error.
@reyoung reyoung force-pushed the feature/remove_error_in_paddle branch from 5e9f67b to f4e57b4 Compare July 27, 2017 02:07
@reyoung reyoung merged commit 89512df into PaddlePaddle:develop Jul 27, 2017
@reyoung reyoung deleted the feature/remove_error_in_paddle branch August 2, 2017 11:35
heavengate pushed a commit to heavengate/Paddle that referenced this pull request Aug 16, 2021
* fix jde negative coords

* update jde model zoo

* describe format for some txt, test=document_fix
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.

2 participants