Skip to content

Create "dummy" supervised models on constant targets #699

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.

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

Merged

Conversation

marcboulle
Copy link
Collaborator

@marcboulle marcboulle commented May 9, 2025

Cela permet de construire un prédicteur, même dans ces cas limites

Mais il restera toujours des cas on peut effectuer la préparation, mais pas la modélisation

  • base vide
  • régression, mais variable cible ne contenant que des valeurs manquantes
  • assez de ressource mémoire pour la préparation, mais pas pour la modélisation
  • pas de modélisation demandée
  • ...

Impacts:

  • KWLearningProblem::ComputeStats
    • lancement de l'apprentissage même si une seule valeur
    • pas de warning, car déjà émis lors de la préparation
  • KWDRNBRegressor::ComputeCumulativeTargetValues, ComputeCumulativeSquareTargetValues
    • pas de lissage dans le cas particulier d'une seule valeur (effet de bord)
    • cela permet de prédire la valeur unique exacte

Tests:

  • LearningTest\TestKhiops\SideEffects\MonoClass: classification
  • LearningTest\TestKhiops\SideEffects\MonoValueR: régression

A noter:

  • un bug de visualisation a été signalé pour la gestion de cet effet de bord, dans le cas de la régression

@marcboulle marcboulle requested a review from popescu-v May 9, 2025 17:06
@marcboulle marcboulle linked an issue May 9, 2025 that may be closed by this pull request
@marcboulle marcboulle force-pushed the 697-consider-creating-dummy-model-on-constant-targets branch from 0e86c9c to ec3564b Compare May 9, 2025 18:20
Copy link
Collaborator

@popescu-v popescu-v left a comment

Choose a reason for hiding this comment

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

See the 2 comments.

@marcboulle marcboulle requested a review from popescu-v May 12, 2025 12:53
Impacts:
- KWLearningProblem::ComputeStats
  - lancement de l'apprentissage meme si une seule valeur
  - pas de warninbg, car deja emis lors de la preparation
- KWDRNBRegressor::ComputeCumulativeTargetValues, ComputeCumulativeSquareTargetValues
  - pas de lissage dans le cas particulier d'un seule valeur (effet de bord)
  - cela permet de predire la valeur unique exacte

Tests:
- LearningTest\TestKhiops\SideEffects\MonoClass: classification
- LearningTest\TestKhiops\SideEffects\MonoValueR: regression
@marcboulle marcboulle force-pushed the 697-consider-creating-dummy-model-on-constant-targets branch from ec3564b to 2f18d0f Compare May 12, 2025 12:54
Copy link
Collaborator

@popescu-v popescu-v left a comment

Choose a reason for hiding this comment

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

LGTM.

@marcboulle marcboulle merged commit 1801d26 into dev May 12, 2025
23 checks passed
@marcboulle marcboulle deleted the 697-consider-creating-dummy-model-on-constant-targets branch May 12, 2025 13:38
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.

Consider Creating "Dummy" Model on Constant Targets
2 participants