Skip to content

Conversation

@giulliadeux
Copy link

This PR introduces changes to the LazyClassifier class. The primary change is the integration of a function, _convert_bool_to_int, to convert boolean columns in the input data to int (0 and 1). This ensures compatibility across classifiers and avoids potential issues that could arise from boolean data types.

Key changes include:

  • Addition of the _convert_bool_to_int function that converts boolean columns to integer values.
  • Modification to the fit method to call the _convert_bool_to_int function for both X_train and X_test datasets.

@nityansuman
Copy link

@giulliadeux I have decided to start mainatnce of this old repo. Please feel free to raise a PR at https://github.com/nityansuman/lazypredict-nightly. I have fixed a couple of issues and plan to extend the model selection and fix some of the issues mentioned here.

@shankarpandala
Copy link
Owner

Closing as this PR is 2 years old without any context about what issue #436 was.

Issue #436 appears to be closed or deleted, making this PR impossible to evaluate. If the fix is still needed, please open a new PR with:

  • Clear description of the problem
  • Reference to a current issue
  • Updated code for the current codebase

Thank you for the contribution.

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.

3 participants