Opened 4 years ago

Closed 3 years ago

#5594 closed defect (fixed)

Remove audio spectrograms (cleanup now unused code)

Reported by: Ben Sturmfels Owned by:
Priority: minor Milestone: 0.11.0
Component: programming Keywords:
Cc: Fernando Gutierrez Parent Tickets:

Description (last modified by Ben Sturmfels)

Audio spectrograms don't work with Python 3 due due to a Python 2-only depedency on audiolab. The Python 2 code us unfortunately rather fragile though, which often results in bad experiences for people setting up MediaGoblin as well as bug reports.

Given we'll soon be switching to Python 3-only, I propose we drop audio spectrograms entirely. This will result in a less attractive but more reliable system.

See also #5457 for background.

Change History (4)

comment:1 by Ben Sturmfels, 4 years ago

Description: modified (diff)

comment:2 by Ben Sturmfels, 4 years ago


I've now disabled spectrograms for Python 2 in 4f3f70d1. This change only disables spectrograms and updates install docs and Dockerfiles. We still need to strip out all the spectrogram code and extlib/freesound/.

Moving this to 0.11.0 so we don't forget to do that cleanup work.

comment:3 by Ben Sturmfels, 4 years ago

Summary: Remove audio spectrogramsRemove audio spectrograms (cleanup now unused code)

comment:4 by Ben Sturmfels, 3 years ago

Cc: Fernando Gutierrez added
Resolution: fixed
Status: newclosed

The Python 2-only spectrogram code and libraries have now been removed thanks to Fernando in #5610. Yay!

Note: See TracTickets for help on using tickets.