Skip to content

Batbot metadata drawing#327

Merged
BryonLewis merged 77 commits intomainfrom
batbot-metadata-drawing
Feb 12, 2026
Merged

Batbot metadata drawing#327
BryonLewis merged 77 commits intomainfrom
batbot-metadata-drawing

Conversation

@BryonLewis
Copy link
Collaborator

@BryonLewis BryonLewis commented Jan 30, 2026

resolves #286

  • Integrates Batbot Metadata into the PulseMetadat model
    • Adds Cuve (list[time,freq]), and Points ([time,freq]) for the Characteristic Freq, Heel and Knee
    • This data is formatted and placed inside of the same PulseMetadata as the contours
  • Renamed the endpoint for the contours to be solely /pulse_contours, as well as the Schema to separate it out
  • Added in a pulse_data that returned the data for compressed segments
  • Client
    • Added a new layer that graphs the line and the points
    • State objects for turning the line on/off as well as changing color and point size, and line width
    • add the duration and range calculations as toggleable
    • the usePulseMetadata is a separate state store for these objects done as a singleton composable much like useStore.

@BryonLewis BryonLewis changed the base branch from main to batbot-contours January 30, 2026 19:49
Base automatically changed from batbot-contours to main February 6, 2026 17:49
@BryonLewis BryonLewis force-pushed the batbot-metadata-drawing branch from 9b27421 to 9478be5 Compare February 6, 2026 18:29
@BryonLewis BryonLewis marked this pull request as ready for review February 6, 2026 18:38
@BryonLewis BryonLewis requested a review from naglepuff February 6, 2026 18:38
Copy link
Collaborator

@naglepuff naglepuff left a comment

Choose a reason for hiding this comment

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

A couple of general comments, and I noticed that the text alignment gets awkward sometimes, but overall this is looking really good.

@BryonLewis BryonLewis requested a review from naglepuff February 10, 2026 18:37
@BryonLewis BryonLewis requested a review from naglepuff February 10, 2026 21:03
Copy link
Collaborator

@naglepuff naglepuff left a comment

Choose a reason for hiding this comment

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

Looks great! Let me know if you have any questions while rebasing/merging to remove the conflicts.

@BryonLewis BryonLewis requested a review from naglepuff February 11, 2026 20:11
@BryonLewis
Copy link
Collaborator Author

I updated the uv lock file to make sure it installs from pypi instead of github.

I've modified the docker-compose.overrride.yml to install the task extras and use a uv_task_cache for the celery container to enable local dev testing.

I asked for a rereview to make sure I didn't mess anything up.

@BryonLewis BryonLewis merged commit 59275a8 into main Feb 12, 2026
6 checks passed
@BryonLewis BryonLewis deleted the batbot-metadata-drawing branch February 12, 2026 19:32
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.

[Vetting UI] Integrate updated spectrogram overlays and measurement metrics

2 participants