AOSSIE-Org / AOSSIE-Org/OpenVerifiableLLM

[FEATURE]: Complete SentencePiece tokenizer — encode, decode, load and input validation

Open
#51 1 comment 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
18
Forks
31
Avg merge
1m
Merged PRs (30d)
2

Description

## Summary

This issue proposes completing the SentencePieceTokenizer introduced in #17. The class currently supports training only. It is missing **encode**(), **decode**(), **load**(), **input validation,** special tokens, and directory creation


## Background

PR #17 introduced a modular tokenizer architecture with SentencePieceTokenizer as one of the two implementations.

However the current SentencePieceTokenizer only covers the training step

A tokenizer that cannot encode or decode text cannot be used in any downstream pipeline task.


## Problems in Current Implementation

### 1. No encode() or decode()

### 2. No load()

### 3. Missing special tokens in training

### 4. No input validation

### 5. save_path directory never created

## Why This Matters

SentencePiece is used by major modern LLMs:

Without a complete SentencePieceTokenizer, the project cannot support cryptographic verification of pipelines
built on any of these models — which represent the majority of current open source LLMs.


## Scope

This is a change touching only:
- sentencepiece_tokenizer.py

No overlap with any existing open PRs.

## Related

- PR #17 — feat: deterministic tokenizer training and config hashing

### Additional Context

_No response_

### Code of Conduct

- [x] I have joined the [Discord server](https://discord.gg/hjUhu33uAn) and will post updates there
- [x] I have searched existing issues to avoid duplicates

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.