Skip to content

Revert "REGR: concat(ints, bools) casting to object #42092" #42553

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

Closed

Conversation

lithomas1
Copy link
Member

Reverts #42541

This fails ArrayManager.

cc @jbrockmendel

@jbrockmendel
Copy link
Member

The solution is to fix it for ArrayManager

@lithomas1
Copy link
Member Author

This is just temporary, so we can get green, while the ArrayManger case is fixed. If the ArrayManager fix is simple, we can always close this and merge that instead. Otherwise, we should probably merge this after a couple of days so that CI remains green, and re-land #42541

@jreback
Copy link
Contributor

jreback commented Jul 16, 2021

ok why don't we wait a day or 2 to see if can fix ArrayManager here.

@jbrockmendel
Copy link
Member

I think this just needs a change in concat_arrays in the not single_dtype case

@lithomas1
Copy link
Member Author

Gonna close since I think an actual fix has been sent in now.

@lithomas1 lithomas1 closed this Jul 17, 2021
@lithomas1 lithomas1 deleted the revert-42541-regr-42092 branch July 17, 2021 00:26
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