Skip to content

Deprecate mae pixio decoder timm - #2006

Open
lorinczszabolcs wants to merge 2 commits into
lightly-ai:masterfrom
lorinczszabolcs:deprecate-mae-pixio-decoder-timm
Open

Deprecate mae pixio decoder timm#2006
lorinczszabolcs wants to merge 2 commits into
lightly-ai:masterfrom
lorinczszabolcs:deprecate-mae-pixio-decoder-timm

Conversation

@lorinczszabolcs

Copy link
Copy Markdown
Contributor

Follow-up to #1978. Now that MaskedVisionTransformerDecoderTIMM covers both, this deprecates the old TIMM decoders, as discussed in the review (#1978 (comment)).

  • MAEDecoderTIMM and PixioDecoderTIMM emit a DeprecationWarning on init pointing to MaskedVisionTransformerDecoderTIMM, and get a docstring note. Both stay functional and exported.
  • Migrated the last first-party consumer (the imagenette benchmark) off MAEDecoderTIMM.
  • Added deprecation tests; marked the existing decoder and equivalence tests to ignore the warning.

@lorinczszabolcs
lorinczszabolcs force-pushed the deprecate-mae-pixio-decoder-timm branch from 4611b13 to f37c5c6 Compare July 28, 2026 20:11
@lorinczszabolcs

Copy link
Copy Markdown
Contributor Author

@gabrielfruet this is the follow-up PR to deprecate MAEDecoderTIMM and PixioDecoderTIMM.

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