update IVORN and VOEvent channels from LVC -> IGWN

fix #272 (closed)

changed the description

marked this merge request as ready

Looks good over here. I was able to make some voevents that had the new nomenclature: https://gracedb-dev1.ligo.org/superevents/S230406a/files/

merged

I strongly suggest holding off on deploying this to production. We haven't inspected GCN Classic yet to determine what impact this will have.

That's fine, it has to live on test and playground first anyway. What's the timeline for when it needs to in production along side the userguide changes?

@leo-singer @alexander.pace As stated by Leo, this one can not be deployed or merged in any version in production without checking the impact on GCN.

@roberto.depietri is it a no-go on playground? who is the point person for communicating the change to GCN?

Judy Racusin. But we're aware of it now. It may take a couple weeks for us to get to this, unfortunately.

@alexander.pace I was just worried that it should not be released in production (in any form) until UsergUide and GCN side are checked. No problem running in playground.

In case people were interested: https://github.com/nasa-gcn/pygcn/issues/37

@alexander.pace I think this was merged prematurely. We will have to revert this change since there are breaking changes on GCN classic end which may not get resolved in time for O4.

ctrl+z

!117 (merged)

mentioned in merge request !117 (merged)

mentioned in issue emfollow/userguide#312 (closed)