-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
TST/CLN: Warnings from pytest 3.1+ #16481
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
Labels
CI
Continuous Integration
good first issue
Testing
pandas testing functions or related to the test suite
Milestone
Comments
Some are caused by not using raw strings in regexes. Here's a (hopefully complete list)
|
TomAugspurger
added a commit
to TomAugspurger/dask
that referenced
this issue
May 24, 2017
Catches a handful of warnings from the dask.dataframe test suite. Remaining warnings fall into a few classes: 1. ResoucreWarnings from cloudpickls (e.g. dask/dataframe/tests/test_arithmetics_reduction.py::test_frame_series_arithmetic_methods) Not sure what to do here. Probably a legitimate concern of dasks? 2. RuntimeWarnings from numpy (e.g. dask/dataframe/tests/test_arithmetics_reduction.py::test_reductions[False] I think dask should catch these, (followup PR) 3. ImportWarning from dependencies importing C code? e.g. dask/dataframe/tests/test_categorical.py::test_categorical_set_index[disk] I have a fix for partd (msgpack). Pandas has some too. Still investigating, may be a Cython issue 4. DeprecationWarning on regexes. e.g. dask/dataframe/tests/test_groupby.py::test_full_groupby_multilevel[grouper4] I think these are all pandas. xref pandas-dev/pandas#16481 (comment) 5. RuntimeWarning from pandas merge, e.g. dask/dataframe/tests/test_multi.py::test_merge_by_multiple_columns[disk-inner] Filing an issue on pandas
TomAugspurger
added a commit
to TomAugspurger/dask
that referenced
this issue
May 24, 2017
Catches a handful of warnings from the dask.dataframe test suite. Remaining warnings fall into a few classes: 1. ResoucreWarnings from cloudpickls (e.g. dask/dataframe/tests/test_arithmetics_reduction.py::test_frame_series_arithmetic_methods) Not sure what to do here. Probably a legitimate concern of dasks? 2. RuntimeWarnings from numpy (e.g. dask/dataframe/tests/test_arithmetics_reduction.py::test_reductions[False] I think dask should catch these, (followup PR) 3. ImportWarning from dependencies importing C code? e.g. dask/dataframe/tests/test_categorical.py::test_categorical_set_index[disk] I have a fix for partd (msgpack). Pandas has some too. Still investigating, may be a Cython issue 4. DeprecationWarning on regexes. e.g. dask/dataframe/tests/test_groupby.py::test_full_groupby_multilevel[grouper4] I think these are all pandas. xref pandas-dev/pandas#16481 (comment) 5. RuntimeWarning from pandas merge, e.g. dask/dataframe/tests/test_multi.py::test_merge_by_multiple_columns[disk-inner] Filing an issue on pandas
TomAugspurger
added a commit
to TomAugspurger/dask
that referenced
this issue
May 31, 2017
Catches a handful of warnings from the dask.dataframe test suite. Remaining warnings fall into a few classes: 1. ResoucreWarnings from cloudpickls (e.g. dask/dataframe/tests/test_arithmetics_reduction.py::test_frame_series_arithmetic_methods) Not sure what to do here. Probably a legitimate concern of dasks? 2. RuntimeWarnings from numpy (e.g. dask/dataframe/tests/test_arithmetics_reduction.py::test_reductions[False] I think dask should catch these, (followup PR) 3. ImportWarning from dependencies importing C code? e.g. dask/dataframe/tests/test_categorical.py::test_categorical_set_index[disk] I have a fix for partd (msgpack). Pandas has some too. Still investigating, may be a Cython issue 4. DeprecationWarning on regexes. e.g. dask/dataframe/tests/test_groupby.py::test_full_groupby_multilevel[grouper4] I think these are all pandas. xref pandas-dev/pandas#16481 (comment) 5. RuntimeWarning from pandas merge, e.g. dask/dataframe/tests/test_multi.py::test_merge_by_multiple_columns[disk-inner] Filing an issue on pandas
mrocklin
pushed a commit
to dask/dask
that referenced
this issue
Jun 7, 2017
* TST/CLN: Catch more warnings Catches a handful of warnings from the dask.dataframe test suite. Remaining warnings fall into a few classes: 1. ResoucreWarnings from cloudpickls (e.g. dask/dataframe/tests/test_arithmetics_reduction.py::test_frame_series_arithmetic_methods) Not sure what to do here. Probably a legitimate concern of dasks? 2. RuntimeWarnings from numpy (e.g. dask/dataframe/tests/test_arithmetics_reduction.py::test_reductions[False] I think dask should catch these, (followup PR) 3. ImportWarning from dependencies importing C code? e.g. dask/dataframe/tests/test_categorical.py::test_categorical_set_index[disk] I have a fix for partd (msgpack). Pandas has some too. Still investigating, may be a Cython issue 4. DeprecationWarning on regexes. e.g. dask/dataframe/tests/test_groupby.py::test_full_groupby_multilevel[grouper4] I think these are all pandas. xref pandas-dev/pandas#16481 (comment) 5. RuntimeWarning from pandas merge, e.g. dask/dataframe/tests/test_multi.py::test_merge_by_multiple_columns[disk-inner] Filing an issue on pandas * CLN: catch warnings for dask/array * TST: Reset warnings for test Someone is globally setting Warnings to `'once'`. This will reset it specifically for the test. Long-term solution is to figure out who is setting that and fix it there. * Skip on py27 * Fixed skip condition, linting * Additional comments on warnings
Warnings at HEAD of master (5b99ae2) for Python 3.6 follow (https://travis-ci.org/pandas-dev/pandas/jobs/291434962#L3029)
|
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
CI
Continuous Integration
good first issue
Testing
pandas testing functions or related to the test suite
New versions of pytest print a useful summary of warnings that aren't caught during the test run.
Here's a log from a couple days ago
Some of these will require updates to the test suite to just use the new behavior. Others should be explicitly caught with
tm.assert_produces_warning
. (I believe the downstream package warnings are already fixed in master for dask and statsmodels)The text was updated successfully, but these errors were encountered: