Closed jbphet closed 5 years ago
Note: I tried uploading the .wav files that capture the effect in an audio recording, but GitHub doesn't support that file type.
I discussed the approach for this with @Ashton-Morris today because I wanted some input from someone with sound engineering experience. He pointed out that having a compressor/limiter at the end of the signal chain is a pretty common process when mixing, so that has given me more confidence that the idea of having a Web Audio DynamicsCompressorNode
as the last element of the signal chain makes sense.
I'm going to go ahead and close this and will create the 1.6 branch and publish with this fix in place.
Reopening - I'd like to get this tested on multiple platforms during the 1.6 RC testing.
@jbphet I'm getting some wacky and cool sounds when I very tightly flam home + end (with keyboard nav).
Heard on Win 10 Chrome. For phetsims/QA/issues/219.
I tried a lot of home-end key flams and didn't hear anything other than fast tone combinations, none of which caused clipping distortion, and all of which seemed entirely reasonable. I also recorded the output and looked at it, and all peaks were well below 0 dBFS. I was testing on Win 10 and Chrome, as was @phet-steele.
@phet-steele - this is a bit of an unusual test case, but if you think that it is enough of a concern to warrant further investigation, please download and install the demo version of "Total Recorder" and make a recording of what you're hearing that seems problematic. You won't be able to attach the file to the GitHub issue, but you can post it on Google Drive and put a link in this issue.
I may be hearing something in Mac 10.14 safari, mostly in the middle slider, but its hard to tell.
This is addressed, but I'll be leaving the issue linked in the comment above for some more scrutiny at a later date. Closing this one.
While working on #180, I thought I was hearing some distortion of the sound, likely due to clipping, if I moved the sliders up and down quickly. I recorded the sound and measured it with an app called Total Recorder, then examined it using Audacity. Indeed, it's clipping. Here are a couple of screenshots that illustrate.
Big picture view - this shows several rapid up-and-down movements of the resistivity slider with the other two sliders in their default positions:
Here is a zoom in on the most clipped portion:
This should probably be addressed, potentially by adding a dynamics compressor somewhere in the signal chain.