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

fix ctc edit distance evaluator can not print information in v2 API. #3844

Merged
merged 3 commits into from
Sep 4, 2017

Conversation

lcy-seso
Copy link
Contributor

@lcy-seso lcy-seso commented Sep 4, 2017

fixes #3802

@lcy-seso lcy-seso changed the title fix ctc edit distance in v2 API. fix ctc edit distance evaluator can not print information in v2 API. Sep 4, 2017

const std::string delimiter(".");
std::string::size_type foundPos = name.find(delimiter, 0);
CHECK(foundPos != std::string::npos);
Copy link
Contributor

Choose a reason for hiding this comment

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

或许split好些,例如
https://github.com/PaddlePaddle/Paddle/blob/develop/paddle/gserver/evaluators/Evaluator.cpp#L783, 然后取分割后的最后一个:

auto it = this->values_.find(buffers[buffers.size() - 1]);

避免name含有多个.

@lcy-seso lcy-seso force-pushed the fix_ctc_evaluator branch 2 times, most recently from 72db117 to f30fa48 Compare September 4, 2017 07:40
qingqing01
qingqing01 previously approved these changes Sep 4, 2017
Copy link
Contributor

@qingqing01 qingqing01 left a comment

Choose a reason for hiding this comment

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

LGTM.

@lcy-seso lcy-seso merged commit 77a15b6 into PaddlePaddle:develop Sep 4, 2017
@lcy-seso lcy-seso deleted the fix_ctc_evaluator branch September 12, 2017 07:52
heavengate pushed a commit to heavengate/Paddle that referenced this pull request Aug 16, 2021
…Paddle#3844)

* [dev] rbox update2 (PaddlePaddle#3828)

* set lr for 4 card as default, and update

* fix error
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.

paddle v2 CTC error在训练日志中不显示
2 participants