Parisson / TimeSide

scalable audio processing framework and server written in Python
https://timeside.ircam.fr/docs/
GNU Affero General Public License v3.0
369 stars 59 forks source link

[server] MultipleObjectsReturned at /timeside/api/processors/aubio_pitch/ #179

Open gnuletik opened 4 years ago

gnuletik commented 4 years ago

https://sandbox.wasabi.telemeta.org/timeside/api/processors/aubio_pitch/

get() returned more than one Processor -- it returned 2!
Tointoin commented 4 years ago

Hi @gnuletik and @yomguy. This bug has been caused by version change of Aubio pitch in this commit.

I indeed set aubio_pitch version to 0.4.7 when I added version field on models.Processor without being sure of the write one to give (depend on witch aubio algo/decoding version we use, right?)

@yomguy has corrected it to 0.4.6 in aubio_pitch.pyplugin, thanks for that.

However, from the point of view of sandbox: a new plugins is now available with same processor id (i.e. pid) but a different version.

I deleted the former aubio_pitch 0.4.7 processor in sandbox's DB as it does not refer to a real plugin anymore so it temporary fixes the bug.

@yomguy, what is the best short term and the best preferable long term solutions to manage processors unicity regarding there pid and version:

Tointoin commented 4 years ago

see #183 for Processor version specification.