Add alerts task
This peels off the work in !852 (merged) related to the alert task specifically (as opposed to plugging the alerts task into the orchestrator).
I've bumped the version of fastavro to be at least 1.6.1, because 1.6.0 adds support for a strict option in the validate function that we use in the validation unit test, and 1.6.1 fixes a bug in that strict option.
Merge request reports
Activity
requested review from @leo-singer
- Resolved by Cody Messick
- Automatically resolved by Cody Messick
added 4 commits
-
1e0fd595...cc4a2174 - 3 commits from branch
emfollow:main
- 25fd307a - Add alerts task
-
1e0fd595...cc4a2174 - 3 commits from branch
- Automatically resolved by Cody Messick
- Automatically resolved by Cody Messick
- Automatically resolved by Cody Messick
- Automatically resolved by Cody Messick
- Automatically resolved by Cody Messick
- Automatically resolved by Cody Messick
- Automatically resolved by Cody Messick
- Resolved by Cody Messick
added 1 commit
- d3edfcc0 - Rework download external event file so its done in a canvas
- Resolved by Deep Chatterjee
added 1 commit
- c14d68df - Change send_after_downloading_skymap from task to function
added 1 commit
- 5a908d12 - Rename send_after_downloading_skymap to download_skymap_and_send_alert, rename
added 1 commit
- 995bbcca - Rename send_after_downloading_skymap to download_skymap_and_send_alert, rename
added 1 commit
- f49fb201 - Move download of external event into _add_external_coinc_to_alert
- Automatically resolved by Cody Messick
- Automatically resolved by Cody Messick
added 1 commit
- ca28187f - Change _create_base_alert_dict from task to function
added 1 commit
- 71cc7172 - Drop task decorator on _create_base_alert_dict
added 1 commit
- 328257b5 - Fix thinko in get_event call in _add_external_coinc_to_alert
- Automatically resolved by Cody Messick
- Automatically resolved by Cody Messick
added 1 commit
- 2a4dbc4e - Rework download_skymap_and_send_alert function to drop identity task. Change
enabled an automatic merge when the pipeline for 864343ce succeeds
mentioned in merge request !852 (merged)