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

Fix CI #704

Merged
merged 7 commits into from
Feb 21, 2025
Merged

Fix CI #704

merged 7 commits into from
Feb 21, 2025

Conversation

Zethson
Copy link
Member

@Zethson Zethson commented Jan 30, 2025

testing

Signed-off-by: Lukas Heumos <[email protected]>
@github-actions github-actions bot added bug Something isn't working enhancement New feature or request labels Jan 30, 2025
Signed-off-by: Lukas Heumos <[email protected]>
Signed-off-by: Lukas Heumos <[email protected]>
@flying-sheep
Copy link
Member

flying-sheep commented Feb 21, 2025

OK, I see:

self = <joblib.parallel.BatchCompletionCallBack object at 0x7f7050dccc50>

    def _return_or_raise(self):
        try:
            if self.status == TASK_ERROR:
>               raise self._result
E               joblib.externals.loky.process_executor.TerminatedWorkerError: A worker process managed by the executor was unexpectedly terminated. This could be caused by a segmentation fault while calling the function or by an excessive memory usage causing the Operating System to kill the worker.
E               
E               The exit codes of the workers are {EXIT(1), EXIT(1)}

/opt/hostedtoolcache/Python/3.12.9/x64/lib/python3.12/site-packages/joblib/parallel.py:763: TerminatedWorkerError
----------------------------- Captured stdout call -----------------------------
! Set smaller span value in the case of a `segmentation fault` error.
! Set larger span in case of svddc or other near singularities error.
Processing data...                                            0% -:--:--
----------------------------- Captured stderr call -----------------------------
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'joblib'
/opt/hostedtoolcache/Python/3.12.9/x64/bin/python: Error while finding module specification for 'joblib.externals.loky.backend.popen_loky_posix' (ModuleNotFoundError: No module named 'joblib')
/opt/hostedtoolcache/Python/3.12.9/x64/bin/python: Error while finding module specification for 'joblib.externals.loky.backend.popen_loky_posix' (ModuleNotFoundError: No module named 'joblib')
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'joblib'
/opt/hostedtoolcache/Python/3.12.9/x64/bin/python: Error while finding module specification for 'joblib.externals.loky.backend.popen_loky_posix' (ModuleNotFoundError: No module named 'joblib')

Of special note is the stderr content:

…/bin/python: Error while finding module specification for joblib.externals.loky.backend.popen_loky_posix (ModuleNotFoundError: No module named joblib)

Since that’s in stderr (and not actually a thrown exception that pytest receives), I assume these come from the workers, which would mean that the workers are executed in an environment that doesn’t have joblib installed or PYTHONPATH set in a way that it doesn‘t contain joblib.

I found this when searching the web for the error message:

@flying-sheep
Copy link
Member

OK, I filed actions/runner-images#11639

Signed-off-by: Lukas Heumos <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Feb 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.71%. Comparing base (cf49b53) to head (122250c).
Report is 10 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #704      +/-   ##
==========================================
+ Coverage   64.97%   65.71%   +0.73%     
==========================================
  Files          46       47       +1     
  Lines        6007     6110     +103     
==========================================
+ Hits         3903     4015     +112     
+ Misses       2104     2095       -9     

@flying-sheep flying-sheep marked this pull request as ready for review February 21, 2025 09:59
@Zethson Zethson merged commit dcae039 into main Feb 21, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants