-
Notifications
You must be signed in to change notification settings - Fork 351
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
Interface changes in genseq #72
Conversation
Conflicts: blocks/graph.py
@@ -73,8 +73,6 @@ class BaseSequenceGenerator(Initializeable): | |||
average of the annotations. | |||
|
|||
* For speech recognition we would have three: the weighted average, | |||
the alignment and the monotonicity penalty. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you want to remove this line from the comment?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, I don't.
What should with this PR, @dmitriy-serdyuk? |
I'm going to contunue the work and port to blocks the beam search I implemented for our inner project. |
Okay, I keep it open. I do not want to rush you, but this is the main missing part of the sequence generation framework (apart that it is badly documented, but I am working on that). So I will have to do it myself if you do not start soon. |
Sorry! Pressed wrong button. |
Changed interface of SequenceGenerator and Emitter for beam search.