Fix the GraceDb upload functionality of PAstroAlertConsumer (!14) · Merge requests · spiir-group / spiir · GitLab

Maintenance will be performed on git.ligo.org, containers.ligo.org, and docs.ligo.org on Tuesday 24 September 2024 starting at approximately 9am PDT. It is expected to take around 30 minutes and there will be several periods of downtime throughout the maintenance. Please address any comments, concerns, or questions to the helpdesk.

Daniel Tang requested to merge fix/debug-upload-gracedb into dev Mar 09, 2023

When processing alerts with the PAstroConsumer from a IGWNAlert topic, the process_alert method would fail when attempting to upload.

This PR fixes #14 (closed).

Changes

Fixes the GraceDb setup method with correct indentation as previous logic was not reachable.
Only attempt to connect to GraceDB with the GraceDB client if upload_gracedb=True.
Update share script to enable a --upload-gracedb flag
Edit log message for payload saving code.

Tests

I have run a test on CIT with my credentials and the issue is resolved.

Edited Mar 09, 2023 by Daniel Tang