Skip to content

Fix multi hypothesis streaming #1496

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

Draft
wants to merge 44 commits into
base: main
Choose a base branch
from

Conversation

zphoenixrises
Copy link
Member

No description provided.

zphoenixrises and others added 30 commits April 10, 2025 17:11
- Cleanup CSV generation to produce only median values
- Switch from streaming to non streaming output
For greedy selection, or if already in a `softmax`/`log_softmax` form don't take temperature,
Otherwise only take temperature on the values that are getting converted to `softmax`
Zeeshan Haque and others added 14 commits April 21, 2025 16:38
… that `check_host_array` operates properly
…o ensure that `check_host_array` operates properly"

This reverts commit 01523de.
[WARNING] This can break stuff. Just meant for parallel iteration.
sfnp.argmax uses XTensor's implementation of argmax underneath,
which is slower than how numpy implements it. This commit changes
the argmax calls in favor of the numpy implementation.
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.

3 participants