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 bug on macOS from pickling a closure #1315

Closed

Conversation

dgerlanc
Copy link
Contributor

Resolves: #1314

@codecov
Copy link

codecov bot commented Nov 27, 2022

Codecov Report

Merging #1315 (3eb3375) into main (71e2f1e) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1315   +/-   ##
=======================================
  Coverage   74.75%   74.75%           
=======================================
  Files         194      194           
  Lines       49899    49899           
  Branches    10554    10554           
=======================================
  Hits        37303    37303           
  Misses      10269    10269           
  Partials     2327     2327           

 - Fix serialization/pickling error by moving `f_build` out of
  `test_cache_race_condition` and to module scope.
 - Add conditional to check for macOS on Python versions < 3.8 and use "spawn" instead of fork
   to launch processes
 - Reduce number of forked processes to 2
 - Reduce number of loop iterations to 2
 - Use constant scaling factor instead of random factors

Resolves: aesara-devs#1314
@dgerlanc dgerlanc force-pushed the fix_test_cache_race_condition branch from aae5979 to 3eb3375 Compare February 18, 2023 14:51
@dgerlanc
Copy link
Contributor Author

Resolved by #1452

@dgerlanc dgerlanc closed this Mar 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

test_cache_race_condition fails on macOS
2 participants