astronomy-commons / hipscat-import

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

Print file name for mapping/splitting failures. #318

Closed delucchi-cmu closed 1 month ago

delucchi-cmu commented 1 month ago

Change Description

Closes #301

Uses dask's distributed print call to log any exception encountered in tasks. This allows for some custom formatting, including the name of the input file, or reference to healpix shard.

Code Quality

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 98.58156% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 99.53%. Comparing base (793212f) to head (fc5bd4b).

Files Patch % Lines
src/hipscat_import/soap/map_reduce.py 96.22% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #318 +/- ## ======================================= Coverage 99.52% 99.53% ======================================= Files 25 25 Lines 1272 1292 +20 ======================================= + Hits 1266 1286 +20 Misses 6 6 ```

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

codecov-commenter commented 1 month ago

Codecov Report

Attention: Patch coverage is 98.62069% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 99.53%. Comparing base (793212f) to head (7064f1a).

Files Patch % Lines
src/hipscat_import/soap/map_reduce.py 96.07% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #318 +/- ## ======================================= Coverage 99.52% 99.53% ======================================= Files 25 25 Lines 1272 1293 +21 ======================================= + Hits 1266 1287 +21 Misses 6 6 ```

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