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

TST: add test for ffill/bfill for non unique multilevel #29763

Merged

Conversation

ganevgv
Copy link
Contributor

@ganevgv ganevgv commented Nov 21, 2019

Copy link
Member

@WillAyd WillAyd left a comment

Choose a reason for hiding this comment

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

Test lgtm but can you move to pandas/tests/groupby/test_transform.py?

@WillAyd WillAyd added the Testing pandas testing functions or related to the test suite label Nov 21, 2019
@WillAyd WillAyd added this to the 1.0 milestone Nov 21, 2019
@ganevgv ganevgv requested a review from WillAyd November 22, 2019 19:50
@@ -911,6 +911,36 @@ def test_pct_change(test_series, freq, periods, fill_method, limit):
tm.assert_frame_equal(result, expected.to_frame("vals"))


def test_ffill_non_unique_multilevel():
Copy link
Member

Choose a reason for hiding this comment

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

Is there an easy way to parametrize on ffill / bfill? Seems like testing both would be nice

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agreed - now testing both ffill and bfill.

@ganevgv ganevgv changed the title TST: add test for ffill for non unique multilevel TST: add test for ffill/bfill for non unique multilevel Nov 23, 2019
@ganevgv ganevgv requested a review from WillAyd November 23, 2019 17:41
@WillAyd WillAyd merged commit 6f1accd into pandas-dev:master Nov 23, 2019
@WillAyd
Copy link
Member

WillAyd commented Nov 23, 2019

Great thanks @ganevgv - loving all of the PRs

keechongtan added a commit to keechongtan/pandas that referenced this pull request Nov 25, 2019
…ndexing-1row-df

* upstream/master: (185 commits)
  ENH: add BooleanArray extension array (pandas-dev#29555)
  DOC: Add link to dev calendar and meeting notes (pandas-dev#29737)
  ENH: Add built-in function for Styler to format the text displayed for missing values (pandas-dev#29118)
  DEPR: remove statsmodels/seaborn compat shims (pandas-dev#29822)
  DEPR: remove Index.summary (pandas-dev#29807)
  DEPR: passing an int to read_excel use_cols (pandas-dev#29795)
  STY: fstrings in io.pytables (pandas-dev#29758)
  BUG: Fix melt with mixed int/str columns (pandas-dev#29792)
  TST: add test for ffill/bfill for non unique multilevel (pandas-dev#29763)
  Changed description of parse_dates in read_excel(). (pandas-dev#29796)
  BUG: pivot_table not returning correct type when margin=True and aggfunc='mean'  (pandas-dev#28248)
  REF: Create _lib/window directory (pandas-dev#29817)
  Fixed small mistake (pandas-dev#29815)
  minor cleanups (pandas-dev#29798)
  DEPR: enforce deprecations in core.internals (pandas-dev#29723)
  add test for unused level raises KeyError (pandas-dev#29760)
  Add documentation linking to sqlalchemy (pandas-dev#29373)
  io/parsers: ensure decimal is str on PythonParser (pandas-dev#29743)
  Reenabled no-unused-function (pandas-dev#29767)
  CLN:F-string in pandas/_libs/tslibs/*.pyx (pandas-dev#29775)
  ...

# Conflicts:
#	pandas/tests/frame/indexing/test_indexing.py
proost pushed a commit to proost/pandas that referenced this pull request Dec 19, 2019
proost pushed a commit to proost/pandas that referenced this pull request Dec 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Testing pandas testing functions or related to the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

df.groupby('symbol')['status'].ffill() results in Exception: cannot handle a non-unique multi-index!
2 participants