Skip to content

Estimator's output_dir does not work with ./ #81

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
popescu-v opened this issue Jul 5, 2023 · 3 comments · Fixed by #367
Closed

Estimator's output_dir does not work with ./ #81

popescu-v opened this issue Jul 5, 2023 · 3 comments · Fixed by #367
Labels
Priority/0-High To do now Status/InDevelopment The issue is in development by one or more team members
Milestone
11.0.0.0

Comments

@popescu-v
Copy link
Collaborator

Description

When specifiying output_dir="./" the estimator does not find the Modeling.kdic and fit fails.

Questions/Ideas

  • Activate trace=True in the train_predictor call to see where are the file put when output_dir="./"
@popescu-v popescu-v added Status/StandBy The issue is on stand-by (usually blocked by external dependencies) Type/Bug labels Jul 5, 2023
@popescu-v
Copy link
Collaborator Author

This is a Khiops issue. Waiting for a resolution in a future version.

@folmos-at-orange folmos-at-orange added the Priority/2-Low To do after P1 label Mar 12, 2024
@folmos-at-orange
Copy link
Member

Khiops 11 will have solved this

@popescu-v popescu-v added this to the 11.0.0.0 milestone Nov 18, 2024
@folmos-at-orange folmos-at-orange mentioned this issue Jan 11, 2025
Closed
@folmos-at-orange
Copy link
Member

Roadmap to resolution:

  • At startup, make the runner set os.environ["KHIOPS_API_MODE"] = "true" (new feature of V11)
  • Eliminate all code related to the function create_unambiguous_khiops_path in core/api.py

@popescu-v popescu-v linked a pull request Mar 18, 2025 that will close this issue
Prepare v11 support #367
Merged
6 tasks
@popescu-v popescu-v mentioned this issue Mar 18, 2025
Merged
6 tasks
@popescu-v popescu-v added Status/InDevelopment The issue is in development by one or more team members Priority/0-High To do now and removed Status/StandBy The issue is on stand-by (usually blocked by external dependencies) Priority/2-Low To do after P1 labels Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Priority/0-High To do now Status/InDevelopment The issue is in development by one or more team members
Projects
None yet
Milestone
11.0.0.0
Development

Successfully merging a pull request may close this issue.

Prepare v11 support KhiopsML/khiops-python
2 participants
@folmos-at-orange @popescu-v