Skip to content

Return logits beamsearch#2025

Open
cmullovisl wants to merge 5 commits intoOpenNMT:masterfrom
cmullovisl:return-logits-beamsearch
Open

Return logits beamsearch#2025
cmullovisl wants to merge 5 commits intoOpenNMT:masterfrom
cmullovisl:return-logits-beamsearch

Conversation

@cmullovisl
Copy link

The current beam search code does not accumulate logits across time steps. This pull request implements tracking of the token logit sequence of the current alive beams and returns the logit sequence for the finally chosen hypotheses.

@cmullovisl
Copy link
Author

It seems one of the CI jobs failed due to issues unrelated to the pull request ("Could not resolve host: github.com"). Could I ask for a re-run of the failed build-and-test-cpp-arm64 (macos-15, ruy) job?

@cmullovisl cmullovisl marked this pull request as ready for review March 9, 2026 15:59
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.

1 participant