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

Dev/bryon spectrogram update #102

Merged
merged 12 commits into from
Apr 19, 2024

Conversation

BryonLewis
Copy link
Collaborator

@BryonLewis BryonLewis commented Apr 12, 2024

  • Moved spectrogram and compressed spectrogram generation into a task. Did this because depending on the audio files this could take long time
  • Creates a CompressSpectrogram model that uses an object storage file for the generated image. Also includes the widths/starts/stops information as well and the ability to do cache_invalidation indicating that another spectrogram needs to be generated because annotations have changed
  • Front-end was switched from utilizing base64 images to using direct references to the object storage URL.
  • Upgraded the pip version because the installation was failing

TODO:

  • Correct the vertical sizing of the areas for the image displays. Right now it isn't sized properly to center the image in the field of view.

@BryonLewis BryonLewis changed the base branch from main to dev/spectrogram-updates April 12, 2024 20:13
@bluemellophone bluemellophone force-pushed the dev/spectrogram-updates branch from 714467b to 0f8f479 Compare April 17, 2024 21:03
@BryonLewis BryonLewis marked this pull request as ready for review April 19, 2024 14:33
@BryonLewis BryonLewis merged commit 7972422 into dev/spectrogram-updates Apr 19, 2024
6 checks passed
BryonLewis added a commit that referenced this pull request May 2, 2024
* Initial spectrogram updates for colormaps and high resolution renders

requirements fix

update spectro

wip

* Local changes for bug fixes, allowing inference mode as well as high-resolution renders

Linting after rebase

Consolidate changes and linting

* Dev/bryon spectrogram update (#102)

* wip

* requirements fix

* update spectro

* compressed spectrogram model

* linting

* change spectrogram to utilzie URLs instead of base64

* update viewer defaults

* setup update, pip upgrade, merged testing

---------

Co-authored-by: Jason Parham <[email protected]>

* enabling tile-server visualization

* recording admin to re-generate commands

---------

Co-authored-by: Jason Parham <[email protected]>
@BryonLewis BryonLewis deleted the dev/bryon-spectrogram-update branch December 17, 2024 17:01
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