Opened 8 years ago

Closed 6 years ago

#731 closed defect (fixed)

Setting create_spectrogram to false breaks processing

Reported by: pythonsnake Owned by:
Priority: minor Milestone: 0.7.0
Component: programming Keywords: audio, processing, sql, bitsized
Cc: Parent Tickets:

Description

I tried disabling spectograms and a bunch of bugs appeared:

  • Trying to submit audio files yield that error:
/home/pythonsnake/repositories/mediadwarf/lib/python2.6/site-packages/SQLAlchemy-0.8.1-py2.6-linux-x86_64.egg/sqlalchemy/types.py:753: SAWarning: Unicode type received non-unicode bind param value.
  return impl_processor(process_param(value, dialect))
  • The thumbnail is set to a nonexistent file,
    /mgoblin_media/fake/thumb/path.jpg
    

Maybe we should rather define a default audio thumbnail for that kind of situation?

Subtickets

Change History (5)

comment:1 Changed 8 years ago by pythonsnake

Keywords: bitsized added
Priority: criticalminor

Edit: first point is irrelevant, and should actually be:

  • .audio-player class has "display: none" and is invisible. So the fastest way is to add a default "pseudo-spectogram" image?
Last edited 8 years ago by pythonsnake (previous) (diff)

comment:2 Changed 7 years ago by Christopher Allan Webber

Milestone: 0.5.00.6.0

comment:3 Changed 7 years ago by Christopher Allan Webber

Milestone: 0.6.00.7.0

comment:4 Changed 7 years ago by Christopher Allan Webber

It sounds like the easiest solution is to disable the disabling of spectrograms :)

That's a bitesized fix! Let's get it into this release if we can!

comment:5 Changed 6 years ago by Christopher Allan Webber

Owner: pythonsnake deleted
Resolution: fixed
Status: newclosed

Okay, pushed up a fix in 7e0c51a which just makes it so that it isn't an option... spectrograms are always created.

Note: See TracTickets for help on using tickets.