Skip to content

Remove target column selector for unsupervised converters#285

Merged
cristian-tamblay merged 8 commits into
developfrom
feat/supervised-unsupervised-converters
Sep 8, 2025
Merged

Remove target column selector for unsupervised converters#285
cristian-tamblay merged 8 commits into
developfrom
feat/supervised-unsupervised-converters

Conversation

@Irozuku
Copy link
Copy Markdown
Collaborator

@Irozuku Irozuku commented Sep 5, 2025

This pull request adds the concept of "supervised" converters to the backend and frontend, ensuring that converters which require a target column (supervised learning) are properly flagged and handled in the UI. It also includes some minor code cleanups and bug fixes.

Backend changes:

  • Added a SUPERVISED attribute to the BaseConverter class and set it to True for relevant scikit-learn and imbalanced-learn converters, such as SelectKBest, SelectFpr, SelectFdr, SelectFwe, SelectPercentile, GenericUnivariateSelect, and the imbalanced-learn wrappers. The metadata dictionary now includes the supervised flag for each converter. [1] [2] [3] [4] [5] [6] [7] [8] [9]
  • Fixed typos in the __init__ methods of SMOTEConverter and SMOTEENNConverter. [1] [2]
  • Minor cleanup of import statements in several scikit-learn converter files. [1] [2] [3] [4] [5]
  • Printed the converter's metadata during instantiation for debugging purposes.
  • Updated ConverterParams to clarify the type of target_index.

Frontend changes:

  • Passed the supervised flag from the converter's metadata to the ScopeStepConverter component. [1] [2]
  • Displayed additional UI elements and tooltips when a converter is supervised, requiring the user to select a target column. The "Next" button is disabled until a target column is selected for supervised converters. [1] [2]

Before (unsupervised converter)

image

After (unsupervised converter)

image

These changes ensure that supervised converters are clearly indicated and that the user experience is improved by requiring the necessary target column input only when appropriate.

@cristian-tamblay cristian-tamblay merged commit bafea02 into develop Sep 8, 2025
4 checks passed
@cristian-tamblay cristian-tamblay deleted the feat/supervised-unsupervised-converters branch September 8, 2025 12:05
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.

2 participants