Skip to content

Update interpretation support to the Khiops Core alpha 10.7.3-a.0#410

Merged
popescu-v merged 3 commits intodevfrom
407-separate-intepretation-dictionary-build-from-the-lever-variables-part
May 28, 2025
Merged

Update interpretation support to the Khiops Core alpha 10.7.3-a.0#410
popescu-v merged 3 commits intodevfrom
407-separate-intepretation-dictionary-build-from-the-lever-variables-part

Conversation

@popescu-v
Copy link
Copy Markdown
Collaborator

@popescu-v popescu-v commented May 21, 2025 •
edited
Loading

closes #407

TODO Before Asking for a Review

  • Rebase your branch to the latest version of dev (or main for release PRs)
  • Make sure all CI workflows are green
  • When adding a public feature/fix: Update the Unreleased section of CHANGELOG.md (no date)
  • Self-Review: Review "Files Changed" tab and fix any problems you find
  • API Docs (only if there are changes in docstrings, rst files or samples):
    • Check the docs build without warning: see the log of the API Docs workflow
    • Check that your changes render well in HTML: download the API Docs artifact and open index.html
    • If there are any problems it is faster to iterate by building locally the API Docs

@popescu-v popescu-v linked an issue May 21, 2025 that may be closed by this pull request
Separate Intepretation Dictionary Build from the Lever Variables Part #407
Closed
@popescu-v popescu-v force-pushed the 407-separate-intepretation-dictionary-build-from-the-lever-variables-part branch from 5d9fa71 to fd7021c Compare May 22, 2025 14:49
@popescu-v popescu-v marked this pull request as ready for review May 22, 2025 15:21
@popescu-v popescu-v force-pushed the 407-separate-intepretation-dictionary-build-from-the-lever-variables-part branch from fd7021c to 0964e97 Compare May 22, 2025 15:26
folmos-at-orange
folmos-at-orange requested changes May 23, 2025
View reviewed changes
@popescu-v popescu-v force-pushed the 407-separate-intepretation-dictionary-build-from-the-lever-variables-part branch from 0964e97 to fc7aa9e Compare May 27, 2025 10:45
popescu-v added 2 commits May 27, 2025 15:06
@popescu-v
Fix typo in sample function output directory
060fff4
@popescu-v
Update interpret_predictor to the latest Core alpha 10.7.3-a.0
0287d7d 
- drop lever variables
- reinstate max_variable_importances
- add global vs. individual variable importance computation suppport
@popescu-v popescu-v force-pushed the 407-separate-intepretation-dictionary-build-from-the-lever-variables-part branch from fc7aa9e to 05ed380 Compare May 27, 2025 13:23
@popescu-v popescu-v force-pushed the 407-separate-intepretation-dictionary-build-from-the-lever-variables-part branch from 05ed380 to 4f0bc4d Compare May 27, 2025 13:47
@popescu-v popescu-v merged commit 9758a3d into dev May 28, 2025
19 checks passed
@popescu-v popescu-v deleted the 407-separate-intepretation-dictionary-build-from-the-lever-variables-part branch May 28, 2025 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@folmos-at-orange folmos-at-orange folmos-at-orange approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Separate Intepretation Dictionary Build from the Lever Variables Part

2 participants

@popescu-v @folmos-at-orange