-
Notifications
You must be signed in to change notification settings - Fork 5
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
Print file name for mapping/splitting failures. #318
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #318 +/- ##
=======================================
Coverage 99.52% 99.53%
=======================================
Files 25 25
Lines 1272 1292 +20
=======================================
+ Hits 1266 1286 +20
Misses 6 6 ☔ View full report in Codecov by Sentry. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #318 +/- ##
=======================================
Coverage 99.52% 99.53%
=======================================
Files 25 25
Lines 1272 1293 +21
=======================================
+ Hits 1266 1287 +21
Misses 6 6 ☔ View full report in Codecov by Sentry. |
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