930364 - standard text message sound is jarring

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::SMS, defect)

Description

[lkenzig]

Phone:
Alcatel One Touch

Firefox OS Version:
1.1.0.0-prerelease

Issue Details:
I get that the text message that is standard is supposed to sound like a boomerang or something, but my first reaction was that I had broken something or it didn't work. This is not the first impression we want to give for one of the first actions people may be trying. Words to describe what I expect to hear are: motion, friendliness, expectation, human (not digital).

Associated App:
Messaging

Comment 1

[Anthony Ricaud (:rik)]

I have to agree. I always feel that the phone is slow when hearing that sound, like it's not able to play the sound smoothly.

Patryk: Do we have another sound in the pipe for this?

Comment 2

[Patryk Adamczyk [:patryk] UX]

I agree. The sounds were all designed last year... Nov 2012, so before we even hit v.1 of the OS let alone knew the performance of the OS, which sounds we needed, could the apps even play those sounds or device speaker quality. We're in the process of getting the vendor to fix some of the sounds and add new ones... if you have another other requests please add them to this etherpad:
https://etherpad.mozilla.org/new-sounds-need

I will take this bug, and hopefully have it fixed v1.3 or v.1.4, this is pending how quickly we can get a contract sorted with the vendor.

Comment 3

[Anthony Ricaud (:rik)]

Nominating for 1.4 because it makes our phone sound slow and broken. Also, it's an easy task on the engineering side.

Patryk: Instead of going with a vendor, have we considered asking community members? A blog post on the planet would probably result in dozens of propositions to choose from.

Comment 4

[Anthony Ricaud (:rik)]

Needinfo for comment 3.

Comment 5

[Patryk Adamczyk [:patryk] UX]

We should have the final sound latest March 5th. But I should be able to add a new work in progress sound in the next couple of weeks or so.

:rik
We use community members for a few things, however sound design is very specialized, we will likely be redoing alot of sounds in the next few releases as part of our refresh. So I will explore that avenue then. I just don't want to take a risk and use a sound from the community with out knowing exactly where that sound came from with these tight timelines especially when we do have a trusted vendor in place.

Currently the vendor is working on a send sound to fit the visual refresh.

Comment 6

[Joe Cheng [:jcheng] (please needinfo)]

hi Patryk, wonder if the final sound is ready? thanks

Comment 7

[Patryk Adamczyk [:patryk] UX]

We're on revision ~20. Hopefully this week!

Comment 8

[Patryk Adamczyk [:patryk] UX]

Attachment: msg_snd.wav

Hey Ben, adding a new send message sound, can you please convert it to opus? Thanks!

Comment 9

[Anthony Ricaud (:rik)]

Attachment: https://github.com/mozilla-b2g/gaia/pull/17578

Ben: I just did |opusenc msg_snd.wav sent.opus|. Let me know if you used other settings.

Comment 10

[Ben Kelly [:bkelly, not reviewing]]

(In reply to Anthony Ricaud (:rik) from comment #9)
> Ben: I just did |opusenc msg_snd.wav sent.opus|. Let me know if you used
> other settings.

That's what I used.  I understand previously the media team hand tuned the options to get the smallest file.  They told me the defaults have gotten better since then, though.  Also, to do the hand-tuning I think they used a tool to detect when audio degradation started to get introduced.  I think we would need their help if we wanted to optimize the file size.

Comment 11

[Julien Wajsberg [:julienw]]

Comment on attachment 8396356 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/17578

r=me but you can probably add one simple test in the initSentAudio suite

Comment 12

[Anthony Ricaud (:rik)]

https://github.com/mozilla-b2g/gaia/commit/42e5cc853b6ae493792583cd8fdef9aea374689b with a test added.

Comment 13

[Carsten Book [:Tomcat]]

hi rik, sorry had to revert this checkin in https://github.com/mozilla-b2g/gaia/commit/a77969711c44555ff55f1d8cb3fdbc9ef0627ef9 for test failure like https://tbpl.mozilla.org/php/getParsedLog.php?id=36788470&tree=B2g-Inbound

Comment 14

[Anthony Ricaud (:rik)]

Paul: In B2G desktop, setting (Audio()).mozAudioChannelType = 'notification' doesn't work. When we set it and then read it, it returns 'normal'. Is that expected?

I could use a mock audio but I'm wondering if there's an underlying bug.

Comment 15

[Paul Adenot (:padenot)]

Yeah, I said that on another bug, we need more plumbing for it to work. I'm happy to provide guidance on how to write the patch.

Comment 16

[Anthony Ricaud (:rik)]

I've used a mock for Audio to get this working on B2G desktop.

https://github.com/mozilla-b2g/gaia/commit/69bb85962ea9c1a6125c0db36ec307d5eaa57e28