Skip to content

Add phi adapter tests#1448

Closed
Canonik wants to merge 1 commit into
TransformerLensOrg:devfrom
Canonik:add-phi-adapter-tests
Closed

Add phi adapter tests#1448
Canonik wants to merge 1 commit into
TransformerLensOrg:devfrom
Canonik:add-phi-adapter-tests

Conversation

@Canonik

@Canonik Canonik commented Jun 28, 2026

Copy link
Copy Markdown

Description

Added focused unit coverage for the PhiArchitectureAdapter under tests/unit/model_bridge/supported_architectures/.

Issue #1302 tracks missing config tests for remaining TransformerBridge architecture adapters. This PR adds coverage for the phi adapter following the existing supported test patterns.

Part of #1302.

No new dependencies are required.

Type of change

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

Screenshots

Not applicable.

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

Validation

  • .venv/bin/python -m pytest tests/unit/model_bridge/supported_architectures/test_phi_adapter.py -q
    • 53 passed
  • .venv/bin/python -m pytest tests/unit/model_bridge/supported_architectures/test_phi3_adapter.py -q
    • 26 passed

@Canonik Canonik changed the base branch from main to dev June 28, 2026 14:19
@Canonik Canonik closed this Jun 28, 2026
@Canonik Canonik deleted the add-phi-adapter-tests branch June 28, 2026 18:37
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