Page MenuHomePhabricator

Move MIDI to audio conversion from Score into TimedMediaHandler
Open, LowPublic

Description

We should move the transcoding of Midi to Vorbis from Score to TMH, so that we can use it for uploaded midi files and reuse it from the Score extension.

Event Timeline

TheDJ created this task.May 18 2016, 9:02 AM
Restricted Application added subscribers: Zppix, Aklapper. · View Herald TranscriptMay 18 2016, 9:02 AM
Stemby added a subscriber: Stemby.Jul 26 2016, 7:13 AM

I think timidity should be replaced by fluidsynth.

$ fluidsynth /usr/share/sounds/sf2/FluidR3_GM.sf2 -T oga -F my_file.ogg my_file.midi

See also issue T50029.

@Stemby: Welcome to Phabricator and thanks for registering an account. Please note that "I think Blue looks better than Red" comments (especially without any evaluation criteria provided) are off-topic here. This task is about moving TiMidity conversion from the Score extension to the TimedMediaHandler extension. Thanks for keeping this task focused.

Stemby added a comment.EditedJul 26 2016, 8:39 AM

OK, sorry to bother you. Read my previous message as «please note that your work risks being lost time, because IMHO timidity should be replaced by fluidsynth». So the right task should be «remove timidity from Score and make MIDI to Vorbis "conversion" in TimedMediaHandler by using fluidsynth».

@Stemby please file separate tasks for separate issues.

Ebe123 renamed this task from Move TiMidity conversion from Score into TimedMediaHandler to Move MIDI to audio conversion from Score into TimedMediaHandler.Nov 19 2018, 2:19 AM

Change 511015 had a related patch set uploaded (by Ebe123; owner: Ebe123):
[mediawiki/extensions/TimedMediaHandler@master] Transcode MIDI files to OGG Vorbis and MP3

https://gerrit.wikimedia.org/r/511015

Ebe123 claimed this task.May 18 2019, 11:23 AM

Change 511123 had a related patch set uploaded (by Ebe123; owner: Ebe123):
[mediawiki/extensions/Score@master] Remove audio generation functionality

https://gerrit.wikimedia.org/r/511123

Krinkle added a subscriber: Krinkle.

Provisionally tagging for Tech News per the Gerrit conversation.

Johan added a subscriber: Johan.May 23 2019, 9:30 AM

@Krinkle @Ebe123 Do you know what a rough timeline for this hitting production might be?

Also, this means MIDI files no longer need the score extension to work, right? So something like the text below?

MIDI files can soon be played without the Score extension. You can then add them with [[File:Filename.midi]]. override_midi and override_audio will no longer work. Instead you need to add the MIDI file below the music score.

@Johan: I wouldn't have much of a timeline for this... I'll try adding tests and making sure mp3 transcoding works well (as it differs from MediaWiki-extensions-Score 's way)

You are correct. However, removing the override_midi and override_audio from MediaWiki-extensions-Score would come later (with https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Score/+/511123), to give some time for deprecation.

@Ebe123 OK! When you're ready, feel free to go to https://phabricator.wikimedia.org/tag/user-notice/ and drag this ticket from "Not ready to announce" to "Announce in next Tech/News" and ping me here, and we'll make sure the communities get to know what's happening. (:

Change 513015 had a related patch set uploaded (by Brion VIBBER; owner: Brion VIBBER):
[mediawiki/vagrant@master] Add MW-Vagrant support for MIDI in TimedMediaHandler

https://gerrit.wikimedia.org/r/513015

Change 514960 had a related patch set uploaded (by Brion VIBBER; owner: Brion VIBBER):
[operations/mediawiki-config@master] Specify the fluidsynth paths for TMH MIDI conversion

https://gerrit.wikimedia.org/r/514960

Change 514962 had a related patch set uploaded (by Brion VIBBER; owner: Brion VIBBER):
[operations/puppet@production] List deps for MIDI to Ogg/MP3 conversion for video scalers

https://gerrit.wikimedia.org/r/514962

brion added a comment.Thu, Jun 6, 9:31 PM

@Johan @Ebe123 I've +2'd the latest patchset and added some configuration patchsets for production and MediaWiki-Vagrant. :) This should be ready to roll soon if no problems in testing.

Change 511015 merged by jenkins-bot:
[mediawiki/extensions/TimedMediaHandler@master] Transcode MIDI files to OGG Vorbis and MP3

https://gerrit.wikimedia.org/r/511015

Change 514960 merged by jenkins-bot:
[operations/mediawiki-config@master] Specify the fluidsynth paths for TMH MIDI conversion

https://gerrit.wikimedia.org/r/514960

Mentioned in SAL (#wikimedia-operations) [2019-06-06T23:57:26Z] <jforrester@deploy1001> Synchronized wmf-config/CommonSettings.php: Specify the fluidsynth paths for TMH MIDI conversion T135597 (duration: 00m 47s)

Change 515000 had a related patch set uploaded (by Ebe123; owner: Ebe123):
[mediawiki/extensions/Score@master] Add tracking categories to pages overriding files

https://gerrit.wikimedia.org/r/515000

TheDJ added a comment.Fri, Jun 7, 6:25 AM

Thanks for the work @Ebe123 !

Change 513015 merged by jenkins-bot:
[mediawiki/vagrant@master] Add MW-Vagrant support for MIDI in TimedMediaHandler

https://gerrit.wikimedia.org/r/513015

Great stuff @Ebe123 !
I'm going to leave this out of Tech News this week, because there's no deployment train next week, so the new features won't be available until the week of 17 June. That way when it is announced in Tech News, the feature will be only 3 days from release instead of 10 days away. :-)