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/spectrogram updates #93

Merged
merged 5 commits into from
May 2, 2024
Merged

Dev/spectrogram updates #93

merged 5 commits into from
May 2, 2024

Conversation

BryonLewis
Copy link
Collaborator

@BryonLewis BryonLewis commented Apr 1, 2024

Bryon's TODO:

  • Verify all previous features work in the system with the new updates and endpoints
  • Clean up the spectrogram generation to be more explicitly and only done in Tasks. I want them in tasks because I'm bit worried about scaling and having large audio files processed in a request. I think it may be better to make it so generation happens only in a task and removing the property reference generation of a spectrogram
  • I believe that CompressedSpecotrgrams should be their own model that have their length, and the start/stop times as well as the duration associated with them. This makes it easer to cache them and invalidate the cache as users add more annotations to the system.
  • Work on converting the base view to utilizing the django large image tile server instead of the base64 version
  • Switch the system of storing the image to the S3 endpoint instead of utilizing the base64 version
  • Going to create an admin task to generate the spectrograms automatically so I can perform the migration that way on the system instead of doing it through a regular migration. This gives me a bit more control.

…resolution renders

Linting after rebase

Consolidate changes and linting
@bluemellophone bluemellophone force-pushed the dev/spectrogram-updates branch from 714467b to 0f8f479 Compare April 17, 2024 21:03
BryonLewis and others added 3 commits April 19, 2024 10:33
* 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]>
@BryonLewis BryonLewis merged commit 829541c into main May 2, 2024
6 checks passed
@BryonLewis BryonLewis deleted the dev/spectrogram-updates 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