Skip to content

Tighten local tokenizer integrity validation #4

Description

@acari-git

Problem

FileSystemLocalModelIntegrityChecker now rejects a model with no tokenizer data, but it accepts any single recognized tokenizer-related file. This can produce false-ready results for tokenizer formats that require a pair, such as vocab.json + merges.txt.

Current verification (2026-08-15)

  • Missing tokenizer is correctly rejected.
  • tokenizer.model, SentencePiece/Tiktoken-style files, vocab.json, and merges.txt are recognized.
  • The checker does not currently validate required combinations such as BPE vocab + merges.
  • LocalModelIntegrityCheckerTests pass.

Desired behavior

  • Accept complete supported tokenizer layouts.
  • Require paired files where the format requires them.
  • Reject empty tokenizer data files.
  • Keep Hugging Face IDs outside local filesystem inspection.
  • Add tests for Fast Tokenizer, SentencePiece, BPE pair, Tiktoken, missing pair member, and empty tokenizer data.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions