Skip to content

TST: Skip flaky, crashing test in Windows #45786

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 3 commits into from
Feb 3, 2022

Conversation

mroeschke
Copy link
Member

Discovered while working on #45478

Logs: https://dev.azure.com/pandas-dev/e01c9671-576d-40f3-a1a1-7bef78604087/_apis/build/builds/73534/logs/54

2022-02-02T19:12:27.1410114Z pandas/tests/test_sorting.py::TestSafeSort::test_codes_out_of_bound[-1] Windows fatal exception: access violation
2022-02-02T19:12:27.1411407Z 
2022-02-02T19:12:27.1414872Z Thread 0x000011cc (most recent call first):
2022-02-02T19:12:27.1417026Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\execnet\gateway_base.py", line 400 in read
2022-02-02T19:12:27.1418547Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\execnet\gateway_base.py", line 432 in from_io
2022-02-02T19:12:27.1420140Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\execnet\gateway_base.py", line 967 in _thread_receiver
2022-02-02T19:12:27.1424677Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\execnet\gateway_base.py", line 220 in run
2022-02-02T19:12:27.1426225Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\execnet\gateway_base.py", line 285 in _perform_spawn
2022-02-02T19:12:27.1427290Z 
2022-02-02T19:12:27.1428448Z Current thread 0x00001088 (most recent call first):
2022-02-02T19:12:27.1434554Z   File "D:\a\1\s\pandas\core\array_algos\take.py", line 163 in _take_nd_ndarray
2022-02-02T19:12:27.1435976Z   File "D:\a\1\s\pandas\core\array_algos\take.py", line 117 in take_nd
2022-02-02T19:12:27.1438483Z   File "D:\a\1\s\pandas\core\algorithms.py", line 1771 in safe_sort
2022-02-02T19:12:27.1441486Z   File "D:\a\1\s\pandas\tests\test_sorting.py", line 434 in test_codes_out_of_bound
2022-02-02T19:12:27.1447616Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\_pytest\python.py", line 183 in pytest_pyfunc_call
2022-02-02T19:12:27.1450037Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\pluggy\_callers.py", line 39 in _multicall
2022-02-02T19:12:27.1455665Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\pluggy\_manager.py", line 80 in _hookexec
2022-02-02T19:12:27.1458131Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\pluggy\_hooks.py", line 265 in __call__
2022-02-02T19:12:27.1461448Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\_pytest\python.py", line 1641 in runtest
2022-02-02T19:12:27.1466555Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\_pytest\runner.py", line 162 in pytest_runtest_call
2022-02-02T19:12:27.1474703Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\pluggy\_callers.py", line 39 in _multicall
2022-02-02T19:12:27.1477211Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\pluggy\_manager.py", line 80 in _hookexec
2022-02-02T19:12:27.1479949Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\pluggy\_hooks.py", line 265 in __call__
2022-02-02T19:12:27.1483707Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\_pytest\runner.py", line 255 in <lambda>
2022-02-02T19:12:27.1487056Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\_pytest\runner.py", line 311 in from_call
2022-02-02T19:12:27.1490846Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\_pytest\runner.py", line 254 in call_runtest_hook
2022-02-02T19:12:27.1496618Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\_pytest\runner.py", line 215 in call_and_report
2022-02-02T19:12:27.1498412Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\_pytest\runner.py", line 126 in runtestprotocol
2022-02-02T19:12:27.1501905Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\_pytest\runner.py", line 109 in pytest_runtest_protocol
2022-02-02T19:12:27.1503892Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\pluggy\_callers.py", line 39 in _multicall
2022-02-02T19:12:27.1505686Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\pluggy\_manager.py", line 80 in _hookexec
2022-02-02T19:12:27.1508520Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\pluggy\_hooks.py", line 265 in __call__
2022-02-02T19:12:27.1515729Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\xdist\remote.py", line 110 in run_one_test
2022-02-02T19:12:27.1517749Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\xdist\remote.py", line 91 in pytest_runtestloop
2022-02-02T19:12:27.1520191Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\pluggy\_callers.py", line 39 in _multicall
2022-02-02T19:12:27.1522404Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\pluggy\_manager.py", line 80 in _hookexec
2022-02-02T19:12:27.1524427Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\pluggy\_hooks.py", line 265 in __call__
2022-02-02T19:12:27.1526201Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\_pytest\main.py", line 323 in _main
2022-02-02T19:12:27.1528352Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\_pytest\main.py", line 269 in wrap_session
2022-02-02T19:12:27.1531223Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\_pytest\main.py", line 316 in pytest_cmdline_main
2022-02-02T19:12:27.1533340Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\pluggy\_callers.py", line 39 in _multicall
2022-02-02T19:12:27.1536051Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\pluggy\_manager.py", line 80 in _hookexec
2022-02-02T19:12:27.1538727Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\pluggy\_hooks.py", line 265 in __call__
2022-02-02T19:12:27.1541284Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\xdist\remote.py", line 291 in <module>
2022-02-02T19:12:27.1543577Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\execnet\gateway_base.py", line 1084 in executetask
2022-02-02T19:12:27.1545576Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\execnet\gateway_base.py", line 220 in run
2022-02-02T19:12:27.1547427Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\execnet\gateway_base.py", line 285 in _perform_spawn
2022-02-02T19:12:27.1549546Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\execnet\gateway_base.py", line 267 in integrate_as_primary_thread
2022-02-02T19:12:27.1553621Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\execnet\gateway_base.py", line 1060 in serve
2022-02-02T19:12:27.1555282Z   File "C:\Miniconda\envs\pandas-dev\lib\site-packages\execnet\gateway_base.py", line 1554 in serve
2022-02-02T19:12:27.1556281Z   File "<string>", line 8 in <module>
2022-02-02T19:12:27.1558066Z   File "<string>", line 1 in <module>
2022-02-02T19:12:27.1604672Z 
2022-02-02T19:12:27.1606666Z [gw0] node down: Not properly terminated
2022-02-02T19:12:27.3719566Z [gw0] FAILED pandas/tests/test_sorting.py::TestSafeSort::test_codes_out_of_bound[-1] 
2022-02-02T19:12:27.3723006Z 
2022-02-02T19:12:27.3724654Z replacing crashed worker gw0

@@ -424,6 +426,11 @@ def test_codes(self, verify, codes, exp_codes, na_sentinel):
tm.assert_numpy_array_equal(result, expected)
tm.assert_numpy_array_equal(result_codes, expected_codes)

@pytest.mark.skipif(
is_platform_windows(),
Copy link
Member

Choose a reason for hiding this comment

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

should do the os.environ("CI") check?

individually the skips you've been adding over the last couple weeks all make sense, but added up they make me nervous that we're losing non-trivial coverage

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah seems reasonable (could make an easy skip for this too)

Copy link
Member Author

Choose a reason for hiding this comment

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

Yup make sense (I'll follow up by making a pandas.compat.is_ci_environment function)

@jreback jreback added Testing pandas testing functions or related to the test suite Windows Windows OS labels Feb 3, 2022
@jreback jreback added this to the 1.4.1 milestone Feb 3, 2022
@jreback
Copy link
Contributor

jreback commented Feb 3, 2022

lgtm merge on green (and backport)

@mroeschke mroeschke merged commit 7651c08 into pandas-dev:main Feb 3, 2022
@mroeschke mroeschke deleted the tst/windows_access_violations branch February 3, 2022 19:05
@mroeschke
Copy link
Member Author

@meeseeksdev backport 1.4.x

@lumberbot-app
Copy link

lumberbot-app bot commented Feb 3, 2022

Something went wrong ... Please have a look at my logs.

jreback pushed a commit that referenced this pull request Feb 4, 2022
phofl pushed a commit to phofl/pandas that referenced this pull request Feb 14, 2022
* TST: Skip flaky, crashing test in Windows

* Add CI check
yehoshuadimarsky pushed a commit to yehoshuadimarsky/pandas that referenced this pull request Jul 13, 2022
* TST: Skip flaky, crashing test in Windows

* Add CI check
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 Windows Windows OS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants