Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Annotation Viewer/Editor #13

Merged
merged 2 commits into from
Jan 10, 2024
Merged

Annotation Viewer/Editor #13

merged 2 commits into from
Jan 10, 2024

Conversation

BryonLewis
Copy link
Collaborator

@BryonLewis BryonLewis commented Jan 8, 2024

Client-side annotation viewer/editor implementation

TODO:

  • Add SpectroInfo properties to the spectrogram endpoint. This provides the width/height of the image plus the start/end points for the time and the frequencies. Also may want to include the freq scale for rendering the annotations differently in the future. For now lets just go with a standard linear scale.
  • Implement API calls for getting annotations from the server, this includes the GET/PUT/PATCH endpoints. Possible the DELETE endpoint as well
  • testing and drawing of a simple annotation on the image utilizing the new LayerManager and Rectangle Layer. Create a simple sample annotation and ensure it can be drawn on the image and viewed properly.
  • Implement the callback events for selecting annotations and editing annotations.

Update 20240110:
Going to merge as is (with the stubbing out of the EditAnnotations) so that we have something for rebasing for adding the spectrogram.

I'll move remaining tasks into a new PR.

@BryonLewis
Copy link
Collaborator Author

Having an issue where I can draw an annotation fill but can draw an outline for some reason. Not sure what is happening with it.

@BryonLewis BryonLewis merged commit 166b35e into main Jan 10, 2024
6 checks passed
This was referenced Jan 10, 2024
@BryonLewis BryonLewis deleted the annotation-viewer branch February 22, 2024 15:50
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.

1 participant