search

scientific-python / lazy-loader

Populate library namespace without incurring immediate import costs
BSD 3-Clause "New" or "Revised" License
116 stars 19 forks source link

Make `lazy_load.load` partially thread-safe #90

Closed stefanv closed 5 months ago

stefanv commented 5 months ago

Attempted to close #88, but closed another race condition instead.

codecov-commenter commented 5 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (bf82b68) 93.95% compared to head (d56c717) 94.44%.

Files Patch % Lines
lazy_loader/__init__.py 95.65% 1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #90 +/- ## ========================================== + Coverage 93.95% 94.44% +0.48% ========================================== Files 4 5 +1 Lines 182 198 +16 ========================================== + Hits 171 187 +16 Misses 11 11 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

stefanv commented 5 months ago

@effigies This may still be worth getting in; it solves the problem when lazy loading, e.g., numpy in parallel.