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

Minor refactoring, resolve conflicts with main #4

Draft
wants to merge 163 commits into
base: enc_dec_t5
Choose a base branch
from

Conversation

afeldman-nm
Copy link

No description provided.

cloudhan and others added 30 commits March 2, 2024 14:34
Co-authored-by: ElizaWszola <[email protected]>
Co-authored-by: Michael Goin <[email protected]>
rkooo567 and others added 30 commits March 21, 2024 08:49
…oder mode; removed encoder/decoder argument of Sequence
…on of relative position encoding based on packed-variable-length-sequences
…ct T5 inference result. Nothing is broken by this commit, unless there is a subsequent commit with changes in order to pass regression tests.
…ks wrong though. Added not_causal option for attn_bias to kernel interface contracts; also switched to batch size 1 to avoid incorrectness likely caused by packed-variable-sequence-length mask having zeroes rather than -inf's
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.