Vocabulary Pruning Engine for Multilingual GLiNER Models - #366
Open
ALI-AL-MARJANI wants to merge 17 commits into
Open
Vocabulary Pruning Engine for Multilingual GLiNER Models#366ALI-AL-MARJANI wants to merge 17 commits into
ALI-AL-MARJANI wants to merge 17 commits into
Conversation
added 11 commits
May 18, 2026 15:47
- scripts/prune_gliner_vocab.py: prune multilingual GLiNER vocab to target language
- scripts/validate_pruned_model.py: 3-tier validator (PASS/SCORE_DRIFT/ENTITY_FAIL),
--score_tol flag
- docs/vocab_pruning.md: full documentation page with benchmark results
- docs/index.md: add vocab_pruning to toctree
- gliner/modeling/encoder.py: fix token_lengths kwarg leak in BiEncoder.encode_labels
- gliner/model.py: wrap onnxruntime import in try/except for ARM compatibility
Benchmarked on urchade/gliner_multi-v2.1 (mDeBERTa-v3, 250k vocab):
English Wikipedia corpus -> 250,105 -> 90,840 tokens (63.7% reduction)
Model size: 1155.8 -> 666.5 MB (42.3% smaller), ALL PASS entity correctness
Collaborator
|
@ALI-AL-MARJANI , thanks for your contributions. Please, fix ruff errors first so we can move forward. |
Fix 25 ruff errors blocking the lint job requested by maintainer: - Sort import blocks (I001) and __all__ (RUF022) across 9 files - Merge duplicate get_train_dataloader (F811) — curriculum path integrated into the single authoritative implementation - Add missing docstring args (D417) in config.py, long_doc.py, model.py - Remove unused imports (F401): math, torch, Dict, numpy - Fix for-loop variable overwrite (PLW2901) in descriptions.py - Remove unused neg_mask variable (F841) in loss_functions.py - Fix EN dash in string literal (RUF001) in trainer.py - Fix docstring summary blank line (D205) in hard_negatives.py - Break line > 120 chars (E501) in model.py Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Author
|
CI should pass now i guess |
Collaborator
|
@ALI-AL-MARJANI , please merge from main as there are some conflicts right now that need to be resolved |
…/vocab-pruning-engine # Conflicts: # README.md # gliner/model.py # gliner/modeling/encoder.py
Author
|
conflicts with main resolved |
train_ablation.py depends on TrainingArguments(loss_type=, dice_gamma=, use_span_width_weight=) and span_dice_loss, which only exist on feat/focal-dice-loss-openvino. Running it from this branch raises TypeError: unexpected keyword argument — document why up front.
Collaborator
|
@ALI-AL-MARJANI , please, resolve also ruff error, thanks. |
CI's unpinned `pip install ruff` picked up 0.16.0, which added PLR0917 and
flagged 4 pre-existing large-API-surface functions (GLiNERConfig.__init__,
three from_pretrained-style methods in model.py) that were never touched by
this branch's own diff. PLR0913 ("too many arguments") is already
deliberately ignored here for the same reason (see max-args=20 in
[tool.ruff.lint.pylint]); PLR0917 is the same category of complaint for the
positional-only subset, so it gets the same treatment.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
ALI-AL-MARJANI
pushed a commit
to ALI-AL-MARJANI/GLiNER-Robust
that referenced
this pull request
Jul 24, 2026
Same root cause as urchade#366: CI's unpinned `pip install ruff` picked up 0.16.0, which added PLR0917 and flagged 4 pre-existing large-API-surface functions (GLiNERConfig.__init__, three from_pretrained-style methods in model.py) untouched by this branch's own diff. PLR0913 is already deliberately ignored here for the same reason (max-args=20 in [tool.ruff.lint.pylint]); PLR0917 is the same category for the positional-only subset. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…runing-engine # Conflicts: # gliner/__init__.py
Pre-existing on upstream/main (PR urchade#371, landed today) -- B028 wasn't triggering before ruff CI silently picked up 0.16.0 (same root cause as the PLR0917 fix). Matches the stacklevel=2 convention already used everywhere else warnings.warn is called in this file. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Author
|
No conflicts with main now and ruff clean |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Multilingual GLiNER models (mDeBERTa-v3) carry a 250k-token embedding matrix.
For single-language deployments, >60% of these embeddings are never accessed.
This creates an unnecessary memory and cold-start bottleneck for edge/CPU deployments.
Solution
scripts/prune_gliner_vocab.pytokenises a target-language corpus, identifies theactive token intersection, slices
word_embeddings.weight, rebuildstokenizer.json,and saves a fully self-contained pruned model loadable via
GLiNER.from_pretrained().Benchmark —
urchade/gliner_multi-v2.1(English Wikipedia, 100k articles)Files changed
scripts/prune_gliner_vocab.py— pruning engine (new)scripts/validate_pruned_model.py— 3-tier correctness validator (new)docs/vocab_pruning.md— documentation with benchmarks (new)docs/index.md— added to toctreegliner/modeling/encoder.py— bugfix:token_lengthskwarg leaked into BiEncoderlabels encoder forward pass
gliner/model.py— bugfix: bareimport onnxruntimereplaced with try/exceptUsage
python scripts/prune_gliner_vocab.py \ --model_id urchade/gliner_multi-v2.1 \ --dataset_for_vocab wikipedia \ --output_dir ./pruned_en \ --lang en