astronomy-commons / hipscat-import

HiPSCat import - generate HiPSCat-partitioned catalogs
https://hipscat-import.readthedocs.io
BSD 3-Clause "New" or "Revised" License
5 stars 3 forks source link

Update to latest dask to mitigate smoke test failures #286

Closed delucchi-cmu closed 4 months ago

delucchi-cmu commented 4 months ago

Some tests are timing out (https://github.com/astronomy-commons/hipscat-import/actions/runs/8796038393/job/24148597886).

In debugging the issue, I figured it was just time to try to update to the latest dask version. This introduced some flakiness in the client.scatter when repeatedly scattering the same object. We can't reuse the scattered object between compute calls, so instead this forces the id to be unique between the two objects instead of re-using it.

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.35%. Comparing base (7c65700) to head (8b5e4af).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #286 +/- ## ======================================= Coverage 99.35% 99.35% ======================================= Files 24 24 Lines 1234 1234 ======================================= Hits 1226 1226 Misses 8 8 ```

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