From 6dc902ee32482f439e983a6e173f6d27627b74ad Mon Sep 17 00:00:00 2001
From: Gaurav Waratkar <gauravwaratkar24@gmail.com>
Date: Fri, 17 Jan 2025 12:15:48 +0530
Subject: [PATCH] RAVEN log fix to specify both GW and Ext events
---
CHANGES.rst | 6 ++++++
gwcelery/tasks/raven.py | 4 ++--
2 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/CHANGES.rst b/CHANGES.rst
index bd8f63800..b9b895b77 100644
--- a/CHANGES.rst
+++ b/CHANGES.rst
@@ -1,6 +1,12 @@
Changelog
=========
+2.6.1 (unreleased)
+------------------
+
+- RAVEN will now specify which events were involved in the log message when
+ waiting for both GW and external sky maps.
+
2.6.0 (unreleased)
------------------
diff --git a/gwcelery/tasks/raven.py b/gwcelery/tasks/raven.py
index 077ef26b0..e881323fc 100644
--- a/gwcelery/tasks/raven.py
+++ b/gwcelery/tasks/raven.py
@@ -596,9 +596,9 @@ def trigger_raven_alert(coinc_far_dict, superevent, gracedb_id,
preferred_gwevent_id, ext_id))
if missing_skymap:
comments.append('RAVEN: Will only publish GRB coincidence '
- 'if spatial-temporal FAR is present. '
+ 'for {0}-{1} if spatial-temporal FAR is present. '
'Waiting for both sky maps to be available '
- 'first.')
+ 'first.'.format(preferred_gwevent_id, ext_id))
if is_far_negative:
comments.append(('RAVEN: publishing criteria not met for {0}-{1},'
' {2} FAR is negative ({3:.4g})'.format(
--
GitLab