Skip to content

BUG: concat not sorting mixed column names when None is included #47331

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
merged 6 commits into from
Jun 14, 2022

Conversation

phofl
Copy link
Member

@phofl phofl commented Jun 13, 2022

I think this is more sensible than the previous version, where an error was raised when None was included.

@phofl phofl added Reshaping Concat, Merge/Join, Stack/Unstack, Explode Index Related to the Index class or subclasses labels Jun 13, 2022
Copy link
Member

@mroeschke mroeschke left a comment

Choose a reason for hiding this comment

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

Small comments otherwise LGTM

@mroeschke mroeschke added this to the 1.5 milestone Jun 14, 2022
@mroeschke mroeschke merged commit ad84367 into pandas-dev:main Jun 14, 2022
@mroeschke
Copy link
Member

Thanks @phofl

@phofl phofl deleted the 47127_improved branch June 14, 2022 16:51
jbrockmendel pushed a commit to jbrockmendel/pandas that referenced this pull request Jun 14, 2022
…das-dev#47331)

* REGR: concat not sorting columns for mixed column names

* Fix none in columns

* BUG: concat not sorting column names when None is included

* Update doc/source/whatsnew/v1.5.0.rst

Co-authored-by: Matthew Roeschke <[email protected]>

* Add gh reference

Co-authored-by: Matthew Roeschke <[email protected]>
jbrockmendel pushed a commit to jbrockmendel/pandas that referenced this pull request Jun 14, 2022
…das-dev#47331)

* REGR: concat not sorting columns for mixed column names

* Fix none in columns

* BUG: concat not sorting column names when None is included

* Update doc/source/whatsnew/v1.5.0.rst

Co-authored-by: Matthew Roeschke <[email protected]>

* Add gh reference

Co-authored-by: Matthew Roeschke <[email protected]>
mroeschke added a commit that referenced this pull request Jun 15, 2022
…7307)

* ENH: Timestamp +- timedeltalike scalar support non-nano

* ENH: Timestamp.__sub__(datetime) with non-nano

* better exception message

* BUG: concat not sorting mixed column names when None is included (#47331)

* REGR: concat not sorting columns for mixed column names

* Fix none in columns

* BUG: concat not sorting column names when None is included

* Update doc/source/whatsnew/v1.5.0.rst

Co-authored-by: Matthew Roeschke <[email protected]>

* Add gh reference

Co-authored-by: Matthew Roeschke <[email protected]>

* Add run-tests action (#47292)

* Add run-tests action

* Fix

* Fix

* Fix

* Update macos-windows.yml

* Update posix.yml

* Update python-dev.yml

* Update action.yml

* Update macos-windows.yml

* Update posix.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* ENH: Timestamp pickle support non-nano tzaware (#47340)

* ENH: DTA to_pydatetime, time, timetz, date, iter support non-nano

* cast in liboffsets

* mypy fixup

Co-authored-by: Patrick Hoefler <[email protected]>
Co-authored-by: Matthew Roeschke <[email protected]>
Co-authored-by: Jonas Haag <[email protected]>
mroeschke added a commit that referenced this pull request Jun 15, 2022
* ENH: Timestamp +- timedeltalike scalar support non-nano

* ENH: Timestamp.__sub__(datetime) with non-nano

* better exception message

* BUG: concat not sorting mixed column names when None is included (#47331)

* REGR: concat not sorting columns for mixed column names

* Fix none in columns

* BUG: concat not sorting column names when None is included

* Update doc/source/whatsnew/v1.5.0.rst

Co-authored-by: Matthew Roeschke <[email protected]>

* Add gh reference

Co-authored-by: Matthew Roeschke <[email protected]>

* Add run-tests action (#47292)

* Add run-tests action

* Fix

* Fix

* Fix

* Update macos-windows.yml

* Update posix.yml

* Update python-dev.yml

* Update action.yml

* Update macos-windows.yml

* Update posix.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* ENH: Timestamp pickle support non-nano tzaware (#47340)

* ENH: Timestamp.tz_localize support non-nano

Co-authored-by: Patrick Hoefler <[email protected]>
Co-authored-by: Matthew Roeschke <[email protected]>
Co-authored-by: Jonas Haag <[email protected]>
yehoshuadimarsky pushed a commit to yehoshuadimarsky/pandas that referenced this pull request Jul 13, 2022
…das-dev#47331)

* REGR: concat not sorting columns for mixed column names

* Fix none in columns

* BUG: concat not sorting column names when None is included

* Update doc/source/whatsnew/v1.5.0.rst

Co-authored-by: Matthew Roeschke <[email protected]>

* Add gh reference

Co-authored-by: Matthew Roeschke <[email protected]>
yehoshuadimarsky pushed a commit to yehoshuadimarsky/pandas that referenced this pull request Jul 13, 2022
…ndas-dev#47307)

* ENH: Timestamp +- timedeltalike scalar support non-nano

* ENH: Timestamp.__sub__(datetime) with non-nano

* better exception message

* BUG: concat not sorting mixed column names when None is included (pandas-dev#47331)

* REGR: concat not sorting columns for mixed column names

* Fix none in columns

* BUG: concat not sorting column names when None is included

* Update doc/source/whatsnew/v1.5.0.rst

Co-authored-by: Matthew Roeschke <[email protected]>

* Add gh reference

Co-authored-by: Matthew Roeschke <[email protected]>

* Add run-tests action (pandas-dev#47292)

* Add run-tests action

* Fix

* Fix

* Fix

* Update macos-windows.yml

* Update posix.yml

* Update python-dev.yml

* Update action.yml

* Update macos-windows.yml

* Update posix.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* ENH: Timestamp pickle support non-nano tzaware (pandas-dev#47340)

* ENH: DTA to_pydatetime, time, timetz, date, iter support non-nano

* cast in liboffsets

* mypy fixup

Co-authored-by: Patrick Hoefler <[email protected]>
Co-authored-by: Matthew Roeschke <[email protected]>
Co-authored-by: Jonas Haag <[email protected]>
yehoshuadimarsky pushed a commit to yehoshuadimarsky/pandas that referenced this pull request Jul 13, 2022
* ENH: Timestamp +- timedeltalike scalar support non-nano

* ENH: Timestamp.__sub__(datetime) with non-nano

* better exception message

* BUG: concat not sorting mixed column names when None is included (pandas-dev#47331)

* REGR: concat not sorting columns for mixed column names

* Fix none in columns

* BUG: concat not sorting column names when None is included

* Update doc/source/whatsnew/v1.5.0.rst

Co-authored-by: Matthew Roeschke <[email protected]>

* Add gh reference

Co-authored-by: Matthew Roeschke <[email protected]>

* Add run-tests action (pandas-dev#47292)

* Add run-tests action

* Fix

* Fix

* Fix

* Update macos-windows.yml

* Update posix.yml

* Update python-dev.yml

* Update action.yml

* Update macos-windows.yml

* Update posix.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* Update python-dev.yml

* ENH: Timestamp pickle support non-nano tzaware (pandas-dev#47340)

* ENH: Timestamp.tz_localize support non-nano

Co-authored-by: Patrick Hoefler <[email protected]>
Co-authored-by: Matthew Roeschke <[email protected]>
Co-authored-by: Jonas Haag <[email protected]>
gregcaporaso added a commit to gregcaporaso/scikit-bio that referenced this pull request Nov 22, 2022
this seems to be a difference in how pd.concat is sorting the columns, and likely coming up because of the `odd_key` used here (an empty `frozendict`). this may be related to this pandas bug fix that went into pandas 1.5.0: pandas-dev/pandas#47331
ebolyen pushed a commit to ebolyen/scikit-bio that referenced this pull request Dec 8, 2022
this seems to be a difference in how pd.concat is sorting the columns, and likely coming up because of the `odd_key` used here (an empty `frozendict`). this may be related to this pandas bug fix that went into pandas 1.5.0: pandas-dev/pandas#47331
gregcaporaso added a commit to scikit-bio/scikit-bio that referenced this pull request Dec 8, 2022
this seems to be a difference in how pd.concat is sorting the columns, and likely coming up because of the `odd_key` used here (an empty `frozendict`). this may be related to this pandas bug fix that went into pandas 1.5.0: pandas-dev/pandas#47331
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Index Related to the Index class or subclasses Reshaping Concat, Merge/Join, Stack/Unstack, Explode
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants