Skip to content

fix(explainability): support dynamic SHAP log-loss baselines - #65

Merged
jiangxt2 merged 2 commits into
masterfrom
fix-shap-log-loss-and-pr-template
Aug 18, 2026
Merged

fix(explainability): support dynamic SHAP log-loss baselines#65
jiangxt2 merged 2 commits into
masterfrom
fix-shap-log-loss-and-pr-template

Conversation

@jiangxt2

Copy link
Copy Markdown
Owner

Description

This PR fixes SHAP TreeExplainer log_loss explanations with SHAP 0.52.0 and simplifies the Tributo pull request template.

The SHAP fix handles the dynamic per-sample base_values returned by SHAP as callable objects, materializes them with the supplied labels, and fails closed for missing labels, mixed callable values, or unsupported shapes. This prevents the previous float + method failure during output reconstruction.

The PR template now follows Ray's concise Description, Related issues, and Additional information structure. Fixed test and open-source checklists were removed; contributors are prompted to record relevant test results, pending integration validation, user/API changes, dependency review, and known risks when applicable.

Related issues

None.

Additional information

  • User-facing impact: SHAP tree explanations using output_target="log_loss" work with the current locked SHAP version; the template change only affects pull request authoring.
  • Validation:
    • The original real XGBoost regression test passed: 1 passed.
    • tests/explainability/test_shap.py: 9 passed.
    • Ruff check/format, CI test-plan audit, and git diff --check passed.
    • Commit hooks passed for the changed Python files and the template change.
  • Environment limitation: the full scheduled data/training shard and the precheck changed-test stage encountered an intermittent native macOS ARM XGBoost process segmentation fault (exit 139); this was not a pytest assertion failure. The targeted real regression test passed independently.

Signed-off-by: jiangxt2 <jiangxt2@vip.qq.com>
Signed-off-by: jiangxt2 <jiangxt2@vip.qq.com>
@jiangxt2
jiangxt2 marked this pull request as ready for review August 18, 2026 08:55
@jiangxt2
jiangxt2 merged commit 30c4ad0 into master Aug 18, 2026
14 checks passed
@jiangxt2
jiangxt2 deleted the fix-shap-log-loss-and-pr-template branch August 18, 2026 08:55
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