Skip to content

CI: azure windows timeouts #42236

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

Closed
mzeitlin11 opened this issue Jun 25, 2021 · 2 comments · Fixed by #42237 or #42256
Closed

CI: azure windows timeouts #42236

mzeitlin11 opened this issue Jun 25, 2021 · 2 comments · Fixed by #42237 or #42256
Labels
CI Continuous Integration numba numba-accelerated operations Windows Windows OS
Milestone

Comments

@mzeitlin11
Copy link
Member

Seeing lots of timeouts on Windows py38_np18 builds, for example https://dev.azure.com/pandas-dev/pandas/_build/results?buildId=62155&view=logs&j=404760ec-14d3-5d48-e580-13034792878f&t=f81e4cc8-d61a-5fb8-36be-36768e5c561a

The main tests showing up in the slowest durations are the groupby numba tests. Would it make sense to do something similar to #42024 in the groupby conftest to reduce the number of combinations run? Also in that run seeing warnings like

pandas/tests/window/test_numba.py: 14 warnings
  D:\a\1\s\pandas\core\window\numba_.py:74: NumbaPerformanceWarning: 
  The keyword argument 'parallel=True' was specified but no transformation for parallel execution was possible.
  
  To find out why, try turning on parallel diagnostics, see https://numba.pydata.org/numba-doc/latest/user/parallel.html#diagnostics for help.
  
  File "pandas\core\util\numba_.py", line 109:
  
              def impl(data, *_args):
              ^
  
    result[i] = numba_func(window, *args)

so the parallel=True case might not be testing anything. cc @mroeschke

@mzeitlin11 mzeitlin11 added CI Continuous Integration numba numba-accelerated operations Needs Triage Issue that has not been reviewed by a pandas team member labels Jun 25, 2021
@lithomas1 lithomas1 removed the Needs Triage Issue that has not been reviewed by a pandas team member label Jun 25, 2021
@lithomas1 lithomas1 added this to the 1.3 milestone Jun 25, 2021
@mzeitlin11
Copy link
Member Author

On attempt #2 of that run it succeeded in just 36 min, used 2 workers. The attempt #1 that timed out spun up 4 workers, as did the failing run posted in the issue body. Maybe enforcing <= 2 workers would help (could it be some kind of memory issue?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration numba numba-accelerated operations Windows Windows OS
Projects
None yet
3 participants