Skip to content

add support for streaming ANTARES alerts#44

Merged
phycodurus merged 5 commits intoTOMToolkit:mainfrom
SAGUARO-MMA:tom-antares
Feb 11, 2026
Merged

add support for streaming ANTARES alerts#44
phycodurus merged 5 commits intoTOMToolkit:mainfrom
SAGUARO-MMA:tom-antares

Conversation

@griffin-h
Copy link
Contributor

This adds support for streaming ANTARES alerts over Kafka. It's essentially a thin wrapper around the functionality in the antares-client package.

@jchate6 jchate6 self-requested a review February 10, 2026 18:29
@jchate6 jchate6 moved this to Needs Review in TOM Toolkit Feb 10, 2026
@jchate6 jchate6 requested a review from phycodurus February 10, 2026 19:24
Copy link
Member

@phycodurus phycodurus left a comment

Choose a reason for hiding this comment

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

Thanks, @griffin-h !!

@phycodurus phycodurus merged commit 61aacac into TOMToolkit:main Feb 11, 2026
@github-project-automation github-project-automation bot moved this from Needs Review to Merged (to dev) in TOM Toolkit Feb 11, 2026
@phycodurus phycodurus moved this from Merged (to dev) to Released in TOM Toolkit Feb 11, 2026
@jchate6 jchate6 moved this from Released to Needs Review in TOM Toolkit Feb 11, 2026
Copy link
Contributor

@jchate6 jchate6 left a comment

Choose a reason for hiding this comment

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

I don't think we should require antares_client as a dependency.
Also, I don't think these changes make much sense without TOMToolkit/tom_antares#81.

I think what we should do here is move antares_client or maybe tom_antares to a dependency group in pyproject.toml and add a section in the README about how to install this app for Tom Antares.

@phycodurus
Copy link
Member

I think what we should do here is move antares_client or maybe tom_antares to a dependency group in pyproject.toml and add a section in the README about how to install this app for Tom Antares.

Agreed. This is address in #49

@jchate6 jchate6 moved this from Needs Review to Merged (to dev) in TOM Toolkit Feb 13, 2026
@jchate6 jchate6 deleted the tom-antares branch February 26, 2026 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Merged (to dev)

Development

Successfully merging this pull request may close these issues.

3 participants