Fixed retryable tasks for raven.py; Fixes #84
This merge request is to make much of the raven.py
workflow retryable and in canvas structure is applicable.
The following sentry errors should be addressed from !581 (merged):
Removed call creating this error: https://sentry.io/organizations/ligo-caltech/issues/1164177171/?query=is%3Aunresolved
Block empty search results from being processed: https://sentry.io/organizations/ligo-caltech/issues/1164176937/?environment=playground&environment=production&query=is%3Aunresolved
calculate_coincidence_far
should be executed now as well.
Merge request reports
Activity
added External Triggers label
added 6 commits
-
e3c75f4f...c9dfc693 - 4 commits from branch
emfollow:master
- 68a32ea9 - Fix errors from previous version
- 9e538648 - Add previous merge request
-
e3c75f4f...c9dfc693 - 4 commits from branch
changed title from WIP: Fixed Retryable tasks for raven.py; Fixes #84 (closed) to Fixed retryable tasks for raven.py; Fixes #84 (closed)
mentioned in merge request !581 (merged)
mentioned in issue #99 (closed)
added 1 commit
- 460145af - Fixed retryable tasks for raven.py; Fixes #84 (closed)
added 1 commit
- 3492cd67 - Fixed retryable tasks for raven.py; Fixes #84 (closed)
It looks like these changes were successful based on the log messages in a test event: https://gracedb-playground.ligo.org/superevents/S190817bw/view/
The only Sentry errors were due to lack of sky maps found when running
ligo_fermi_skymaps.combine_skymaps
(a known bug unrelated to these changes that will be fixed in the near future.)Very nice, good work @brandon.piotrzkowski!