Skip to content

Batbot pulse annotations - #521

Open
BryonLewis wants to merge 11 commits into
mainfrom
batbot-pulse-annotations
Open

Batbot pulse annotations#521
BryonLewis wants to merge 11 commits into
mainfrom
batbot-pulse-annotations

Conversation

@BryonLewis

@BryonLewis BryonLewis commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator
  • Adds a configuration flag to the configuration model for creation of pulse annotations
  • This will then create pulse annotations when processing data and importing it in from the batbot curve by extracting the min/max frequency and times. This creates an annotation with no species label
  • Updates some UI to handle selection of pulses and properly centering the camera for both compressed and uncompressed views
  • Adds some significant digits truncation to pulse width and times.

TODO:

  • Update the millsecond accuracy to allow floats instead of just integers for zoomed in and short pulses

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 11, 2026

Copy link
Copy Markdown

Deploying bats-ai with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7de587a
Status: ✅  Deploy successful!
Preview URL: https://9e800ff6.bats-ai.pages.dev
Branch Preview URL: https://batbot-pulse-annotations.bats-ai.pages.dev

View logs

@BryonLewis
BryonLewis force-pushed the batbot-pulse-annotations branch from 8713b65 to 05bac54 Compare June 11, 2026 14:38
@naglepuff
naglepuff force-pushed the batbot-pulse-annotations branch from 05bac54 to 9ebf5ab Compare August 7, 2026 21:17
@naglepuff
naglepuff marked this pull request as ready for review August 7, 2026 21:17
@naglepuff

Copy link
Copy Markdown
Collaborator

@BryonLewis since you're the author of this PR I can't add you as a reviewer, but I've made some updates to allow all users to edit "batbot" created pulse annotations. I'd appreciate if you took a look and let me know your thoughts.

@BryonLewis BryonLewis left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I would resolve the linting errors that are blocking CI.

I have one I think code change and mostly questions or comments that may be resolved in a subsquent PR.

Mostly I think another PR may be needed to determine what happens when a user edits. Currently it creates a new annotation that the user owns without modifying the batbot annotation. That may be enough for the annotation process. The one thing a user can't do is delete an annotation.

Comment thread bats_ai/core/views/recording.py Outdated
Comment thread bats_ai/core/utils/batbot_annotations.py
Comment thread bats_ai/core/views/recording.py
@naglepuff
naglepuff force-pushed the batbot-pulse-annotations branch from 9ebf5ab to 7de587a Compare August 10, 2026 18:30
@naglepuff
naglepuff force-pushed the batbot-pulse-annotations branch from 4379839 to 5188dc1 Compare August 10, 2026 19:13
@naglepuff

Copy link
Copy Markdown
Collaborator

@BryonLewis feel free to take a loot at the most recent changes

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.

2 participants