Skip to content

Commit 444fbf8

Browse files
yossiovadiaclaude
andcommitted
fix(aibrix): revert PII model to ModernBERT configuration from PR #688
The aibrix E2E profile was designed to use ModernBERT for PII detection (as configured in PR #688), but during the merge it was incorrectly changed to use the LoRA PII model which isn't available in the aibrix deployment. This commit reverts the aibrix PII model configuration to use: - model_id: "models/pii_classifier_modernbert-base_presidio_token_model" - use_modernbert: true This restores the original working configuration from PR #688 and allows the aibrix E2E tests to pass while keeping the LoRA PII auto-detection feature available for other profiles that explicitly configure it. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: Yossi Ovadia <yovadia@redhat.com>
1 parent f379f43 commit 444fbf8

File tree

1 file changed

+3
-5
lines changed
  • deploy/kubernetes/aibrix/semantic-router-values

1 file changed

+3
-5
lines changed

deploy/kubernetes/aibrix/semantic-router-values/values.yaml

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -437,13 +437,11 @@ config:
437437
use_cpu: true
438438
category_mapping_path: "models/category_classifier_modernbert-base_model/category_mapping.json"
439439
pii_model:
440-
# Support both traditional (modernbert) and LoRA-based PII detection
441-
# When model_type is "auto", the system will auto-detect LoRA configuration
442-
model_id: "models/lora_pii_detector_bert-base-uncased_model"
443-
model_type: "auto" # Enables LoRA auto-detection
440+
model_id: "models/pii_classifier_modernbert-base_presidio_token_model"
441+
use_modernbert: true
444442
threshold: 0.7
445443
use_cpu: true
446-
pii_mapping_path: "models/lora_pii_detector_bert-base-uncased_model/pii_type_mapping.json"
444+
pii_mapping_path: "models/pii_classifier_modernbert-base_presidio_token_model/pii_type_mapping.json"
447445

448446
keyword_rules:
449447
- name: "thinking"

0 commit comments

Comments
 (0)