Skip to content

Updating filter overview#85

Merged
rknop merged 11 commits intoLSSTDESC:mainfrom
sidratresearch:sidrat-staging
Apr 24, 2026
Merged

Updating filter overview#85
rknop merged 11 commits intoLSSTDESC:mainfrom
sidratresearch:sidrat-staging

Conversation

@jscora
Copy link
Copy Markdown
Contributor

@jscora jscora commented Apr 22, 2026

  • updates to Pitt-Google status
  • minor updates to information
  • added section with links to the LSST alert schema for reference
  • added table of contents at top

Copy link
Copy Markdown
Contributor

@rknop rknop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small detail

Comment thread docs/filters.rst Outdated

This section details how to create new filters at the broker level for FASTDB to subscribe to, for all of the LSST brokers where that is available. Once you have created your filter, let Rob know the broker and the topic name to get FASTDB subscribed to it.

See the section above for the minimum required data that should be sent with the filtered alerts. Additional data that has been added by the broker will be pulled in by FASTDB automatically and stored in the ``diasource_brokerinfo`` table.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's actually two parts to this.

  • Additional data beyond the minimum that comes from the original LSST alerts gets stored in the diasource_extra table.
  • Anything in the alert that was not from the original LSST alert, i.e. anything the broker added, gets stored in the diasource_brokerinfo table.

(Aside: there is probably some nuance here. Right now, the two brokers I'm listening to, Pitt-Google and Fink, are sending me alerts that are exactly the LSST alerts, plus additional fields they added. It's possible some brokers will send different formats, which we can handle, but we have to write the code in brokerconsumer.py to adapt.)

@rknop rknop merged commit 71a64e8 into LSSTDESC:main Apr 24, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants