Skip to content

Fix a RuntimeException error when automatically correcting deprecated data paths (pre v11) in task args - #617

Merged
popescu-v merged 2 commits into
mainfrom
610-error-on-deprecated-path-usage
Sep 1, 2026
Merged

Fix a RuntimeException error when automatically correcting deprecated data paths (pre v11) in task args#617
popescu-v merged 2 commits into
mainfrom
610-error-on-deprecated-path-usage

Conversation

@tramora

@tramora tramora commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Fixes #610


TODO Before Asking for a Review

  • Rebase your branch to the latest version of main (or main-v10)
  • 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

@tramora
tramora force-pushed the 610-error-on-deprecated-path-usage branch from 9f146a3 to 0579e9d Compare August 27, 2026 16:09
@tramora
tramora requested a review from popescu-v August 28, 2026 15:55
@popescu-v
popescu-v force-pushed the 610-error-on-deprecated-path-usage branch 2 times, most recently from 4557f3c to 778421e Compare September 1, 2026 17:45

@popescu-v popescu-v left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Added a commit with a test that _deprecate_legacy_data_path raises the expected warning.

@popescu-v
popescu-v merged commit ed6d756 into main Sep 1, 2026
13 checks passed
@popescu-v
popescu-v deleted the 610-error-on-deprecated-path-usage branch September 1, 2026 18:13
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.

khiops.core.api.train_predictor() trigger RuntimeError when using deprecated data path

2 participants