Skip to content

Conversation

@jbrockmendel
Copy link
Member

@jbrockmendel jbrockmendel commented Nov 21, 2021

  • closes #xxxx
  • tests added / passed
  • Ensure all linting tests pass, see here for how to run them
  • whatsnew entry

@jreback jreback added Missing-data np.nan, pd.NaT, pd.NA, dropna, isnull, interpolate Reshaping Concat, Merge/Join, Stack/Unstack, Explode labels Nov 25, 2021

if axis == 0 and self.ndim == 2 and self.nblocks > 1:
if (
axis == 0
Copy link
Contributor

Choose a reason for hiding this comment

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

this is super complicated can you split into multiple clauses & add comments

Copy link
Member Author

Choose a reason for hiding this comment

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

sure will comment. fortunately we can rip out some of this in 2.0

Copy link
Member Author

Choose a reason for hiding this comment

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

comment added + green

@jreback jreback added this to the 1.4 milestone Nov 25, 2021
@jreback
Copy link
Contributor

jreback commented Nov 25, 2021

user facing?

@jbrockmendel
Copy link
Member Author

user facing?

yes, there's a whatsnew (forgot to check the checkbox until just now)

@jreback jreback merged commit 7a9975f into pandas-dev:master Nov 25, 2021
@jbrockmendel jbrockmendel deleted the ref-diff-axis branch November 26, 2021 04:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Missing-data np.nan, pd.NaT, pd.NA, dropna, isnull, interpolate Reshaping Concat, Merge/Join, Stack/Unstack, Explode

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants