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.

Sign up for GitHub

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

Jump to bottom

Handle 3rd optional input in CTCGreedyDecoderSeqLen evaluate #5669

Merged
merged 3 commits into from
May 24, 2021
Merged

Handle 3rd optional input in CTCGreedyDecoderSeqLen evaluate #5669

merged 3 commits into from
May 24, 2021

Conversation

mikhail-treskin
Copy link

Details:

  • Fix segfault in evaluates_map for CTCGreedyDecoderSeqLen caused by OPs with two inputs only

Tickets:

  • 52561

@mikhail-treskin mikhail-treskin requested a review from a team May 18, 2021 10:10
@openvino-pushbot openvino-pushbot added the category: Core OpenVINO Core (aka ngraph) label May 18, 2021
@mikhail-treskin mikhail-treskin requested a review from iefode May 18, 2021 10:12
@mikhail-treskin mikhail-treskin requested a review from iimironov May 18, 2021 11:42
@mikhail-treskin mikhail-treskin force-pushed the fix_ctc_greedy_evaluate branch from 2bd8103 to f0071a4 Compare May 19, 2021 13:50
@mikhail-treskin mikhail-treskin force-pushed the fix_ctc_greedy_evaluate branch from f0071a4 to 1b7f7b3 Compare May 19, 2021 15:53
@mikhail-treskin mikhail-treskin enabled auto-merge (squash) May 21, 2021 09:59
@evgenytalanin-intel evgenytalanin-intel merged commit 66d9853 into openvinotoolkit:master May 24, 2021
@mikhail-treskin mikhail-treskin deleted the fix_ctc_greedy_evaluate branch May 24, 2021 09:26
yekruglov pushed a commit to yekruglov/openvino that referenced this pull request Jun 7, 2021
@yekruglov
Handle 3rd optional input in CTCGreedyDecoderSeqLen evaluate (openvin…
c05ab58 
…otoolkit#5669)

* Handle 3rd optional input in CTCGreedyDecoderSeqLen evaluate

* Add test for OP without blank_index input

* Fix default blank_index calculation
rnugmanx pushed a commit to rnugmanx/openvino that referenced this pull request Aug 26, 2021
Handle 3rd optional input in CTCGreedyDecoderSeqLen evaluate (openvin…
9036839 
…otoolkit#5669)

* Handle 3rd optional input in CTCGreedyDecoderSeqLen evaluate

* Add test for OP without blank_index input

* Fix default blank_index calculation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ilyachur ilyachur ilyachur left review comments

@iimironov iimironov iimironov approved these changes

@iefode iefode iefode approved these changes

Assignees
No one assigned
Labels
category: Core OpenVINO Core (aka ngraph)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@mikhail-treskin @ilyachur @iefode @iimironov @evgenytalanin-intel @openvino-pushbot