Skip to content

Conversation

@hernandezc1
Copy link
Collaborator

@hernandezc1 hernandezc1 commented Aug 6, 2025

Summary of Changes:

Changed

  • setup_broker/lsst/templates/ps_lsst_lite_smt.yaml
    • Updated functions used to specify which fields will be present in lite LSST alerts. A draft of the LSST alert schema v8_0 is currently available
  • setup_broker/lsst/setup_broker.sh
    • Updated the default value of schema_version to 8.0

Added

  • setup_broker/lsst/templates
    • bq_lsst_alerts_v8_0_schema.json

@hernandezc1 hernandezc1 self-assigned this Aug 6, 2025
@hernandezc1 hernandezc1 changed the title Update SMT to accommodate LSST schema v8_0 Accommodate LSST schema v8_0 Aug 7, 2025
@hernandezc1 hernandezc1 requested a review from troyraen August 7, 2025 17:07
Copy link
Collaborator

@troyraen troyraen left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

Choose a reason for hiding this comment

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

Did you say that you wrote a script that takes a .avsc file and generates this .json file? Do you want to add that to the repo somewhere? Then we could close #239.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes @troyraen! Where should I place that script in the repo?

Copy link
Collaborator

Choose a reason for hiding this comment

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

How about under broker/setup_broker? Either directly there or make a new dir like broker/setup_broker/helper_scripts?

@hernandezc1 hernandezc1 merged commit a7680d3 into develop Aug 21, 2025
6 of 7 checks passed
@hernandezc1 hernandezc1 deleted the u/ch/schema/v8_0 branch August 21, 2025 18:26
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