desy-multimessenger / nuztf

Central repository for scripts handling ZTF Target of Opportunity requests using AMPEL. This includes neutrino, GRB and GW events.
MIT License
1 stars 0 forks source link

BUG? Test compatibility with new ZTF alerts #39

Closed robertdstein closed 2 years ago

robertdstein commented 3 years ago

Describe the bug ZTF alerts will swap from previous detection history to forced photometry history. This might break things, we need to check. The changeover will be in 1 month, and we may have 1 week to play with example data beforehand.

Expected behavior Ideally backwards compatibility should be maintained (i.e the pipeline still works in the same way for old alerts), but the pipeline will also need to be compatible with new alerts. I have no idea how much of an issue this will be, perhaps ampel will take care of everything. It needs to be checked though.

Additional context Co-ordinating with Jakob Nordin and Jakob van Santen is advised.

JannisNe commented 3 years ago

Update: new ZTF alerts will be rolled out in December.

JannisNe commented 3 years ago

Test sample alerts using the ampel alert shaper

JannisNe commented 3 years ago

All this is being worked on in the branch avro_package_update

simeonreusch commented 2 years ago

I don't think we will ever see FP in ZTF alert packages.