Skip to content
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

Replace np.float with np.float64 in one_hot_encoder #1280

Merged
merged 1 commit into from May 8, 2023

Conversation

sarahyurick
Copy link
Contributor

Any background context you want to provide?

NumPy 1.24.0 removed the np.float alias, so we have to replace np.float with np.float64 in one_hot_encoder.

Another option is to replace np.float with Python's float, although the NumPy version may be useful for consistency with NumPy arrays (source).

@beckernick
Copy link
Contributor

cc @weixuanfu , is this something that TPOT would be open to updating to support recent NumPy version?

@gkirgizov
Copy link

Vote for this PR, a required change. Please, merge this

@joseortiz3
Copy link
Contributor

Please, someone with write access merge this. Every time I install tpot I have to change this line (I change it to float and that seems to work as well).

@sarahyurick
Copy link
Contributor Author

sarahyurick commented May 5, 2023

Hi @jay-m-dev or @perib I was wondering if you have the permissions to review and merge this?

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.

None yet

5 participants