Skip to content

CI: check_freq=False #34050

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 11 commits into from
May 9, 2020
Merged

CI: check_freq=False #34050

merged 11 commits into from
May 9, 2020

Conversation

jbrockmendel
Copy link
Member

Aimed at a test that has been periodically failing in the CI for a week or so.

@@ -1140,6 +1141,8 @@ def assert_series_equal(
Whether to compare internal Categorical exactly.
check_category_order : bool, default True
Whether to compare category order of internal Categoricals.
check_freq : bool, default True
Copy link
Member

Choose a reason for hiding this comment

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

move to below versionadded:: 1.0.2 for check_category_order and add versionadded:: 1.1.0

Comment on lines +1334 to +1335
check_freq : bool, default True
Whether to check the `freq` attribute on a DatetimeIndex or TimedeltaIndex.
Copy link
Member

Choose a reason for hiding this comment

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

add versionadded:: 1.1.0

@jreback jreback added Testing pandas testing functions or related to the test suite Frequency DateOffsets labels May 8, 2020
@jreback jreback added this to the 1.1 milestone May 8, 2020
@jreback
Copy link
Contributor

jreback commented May 8, 2020

looks ok, comments by @simonjayhawkins

@jbrockmendel
Copy link
Member Author

comments addressed

Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

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

lgtm merge into green

we should maybe make the default False for backwards compat here (pls open an issue)

@jbrockmendel
Copy link
Member Author

green

@jreback jreback merged commit 3ac34ff into pandas-dev:master May 9, 2020
@jreback
Copy link
Contributor

jreback commented May 9, 2020

k thanks. pls open an issue to change this to check_freq=False; I think this might break downstream, IIRC a comment by @jorisvandenbossche

rhshadrach pushed a commit to rhshadrach/pandas that referenced this pull request May 10, 2020
@jbrockmendel jbrockmendel deleted the cifixteen branch May 12, 2020 02:49
@MaxWinterstein
Copy link

k thanks. pls open an issue to change this to check_freq=False; I think this might break downstream, IIRC a comment by @jorisvandenbossche

Jepp, this broke some code for me. Will open the mentioned issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frequency DateOffsets Testing pandas testing functions or related to the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants