phetsims / mean-share-and-balance

"Mean: Share and Balance" is an educational simulation in HTML5, by PhET Interactive Simulations.
GNU General Public License v3.0
2 stars 1 forks source link

Add Collect and Share Sounds for screen three #206

Closed Ashton-Morris closed 2 months ago

Ashton-Morris commented 2 months ago

I have a mockup video of what we discussed here.

I have also uploaded these sound files.

There are two ways we discussed trying the Share sound:

  1. Just play the MSaBShareSound.mp3 file when the animation starts
  2. Play one of the MSaBShareSoundSeperateNotes files for each apple as it animates. I have up to seven files which creates a major C chord, providing relief from the G7 that plays for the collect animation.
marlitas commented 2 months ago

@jbphet and I talked through a strategy for firing the sound when apples split, and then land:

View is looking for:

marlitas commented 2 months ago

This has been addressed above. @jbphet I didn't quite know what to do with any code that connected a mainGainNode, so I added a TODO about that and some commented out code. Otherwise I think it's sounding good. My one critique may be that the landing sound feels a little late right now, but wanted to get other opinions before really investigating that.

Ashton-Morris commented 2 months ago

In our last meeting we discussed playing three different densities of "share" chords depending on how many apples was shared.

I am assuming that dividing the possible apples into thirds would be enough. So 1 of 1/3 = small, 2 of 1/3 = medium, and. 3 of 1/3 = large in terms of amounts. Here are the sounds I have made for that.

jbphet commented 2 months ago

The initial implementation of this is complete. Assigning to the sound design team for review of the behavior and additionally to @marlitas for code review.

jbphet commented 2 months ago

Reviewed in the 4/30/2024 sound design meeting, and one piece of feedback is that the "share complete" sounds seem late since they are not being triggered until the animation finishes. We'd like to try just a single, delayed play of this sound instead to see if it works better.

Also, @Ashton-Morris would like to try making the various sounds for the shared quantity a bit more distinct.

jbphet commented 2 months ago

I've modified the behavior to trigging the "share compete" sound earlier. I'd like @Ashton-Morris and @emily-phet to review this aspect to see if they are now good with it.

Also - @Ashton-Morris - please assign this issue back to me once we have the changes you wanted to try with the "wider chords" for this sound.

amanda-phet commented 2 months ago

I had the same thought- glad it came up today. The timing is better to me.

Ashton-Morris commented 2 months ago

I am adding these to files to replace the medium and large sounds, hopefully this is subtle enough but more distinct than what we have currently.

jbphet commented 2 months ago

The timing is approved as of today's design meeting, so all that is left is to integrate the new chords.

marlitas commented 2 months ago

New sounds are in. @Ashton-Morris do you want to take a listen before closing?

Ashton-Morris commented 2 months ago

Sounds good - Closing