Skip to content

Feature/prepend bos padding side brdige#1236

Merged
jlarson4 merged 2 commits intodev-3.x-canaryfrom
feature/prepend-bos-padding-side-brdige
Apr 7, 2026
Merged

Feature/prepend bos padding side brdige#1236
jlarson4 merged 2 commits intodev-3.x-canaryfrom
feature/prepend-bos-padding-side-brdige

Conversation

@jlarson4
Copy link
Copy Markdown
Collaborator

@jlarson4 jlarson4 commented Apr 7, 2026

Description

  • Added more explicit warnings if users attempt to use padding_side or prepend_bos on TransformerBridge, explaining why we don't have those built in directly and how to do those things if needed.
  • Improved intermediate hook firing when running in compatibility mode on GatedMLP

Type of change

  • New feature (non-breaking change which adds functionality)

Checklist:

  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have not rewritten tests relating to key interfaces which would affect backward compatibility

@jlarson4 jlarson4 merged commit 519ae6c into dev-3.x-canary Apr 7, 2026
21 checks passed
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