Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

Gain:0 should be off? #35

Closed
grimmdude opened this issue Oct 1, 2016 · 7 comments
Closed

Gain:0 should be off? #35

grimmdude opened this issue Oct 1, 2016 · 7 comments

Comments

@grimmdude
Copy link

Hey man, great library. Both you and I seem to be interested in programming and music. I'm working on a MIDI player and am using this library for the sounds. I find that when I set gain to 0 the note still sounds. Would be be possible for the note to be silent in that case?

Here's the MIDI player I'm working on: http://grimmdude.com/MidiPlayerJS/

Thanks,
Garrett

@danigb
Copy link
Owner

danigb commented Oct 2, 2016

Hi Garret,

Yes, I've seen your before in some music related repo discussion. The midi player project looks great! Congrats.

And yes, { gain: 0 } should be silent sounds. If not, is a bug. I'll take a look shortly.

Thanks,
Dani

@danigb danigb closed this as completed in d140bfe Oct 2, 2016
@danigb
Copy link
Owner

danigb commented Oct 2, 2016

Hi @grimmdude,

I think this is fixed. Please feel free to reopen if doesn't work to you.
Regards,
Dani

@grimmdude
Copy link
Author

I'm not sure if it's related, but after these changes it seems that now sounds are not working at all and I'm seeing a bunch of these errors:

Buffer E6 not found.

I'm using dist/soundfont-player.min.js from master branch and loading the soundfont from https://raw.githubusercontent.com/gleitz/midi-js-soundfonts/gh-pages/MusyngKite/clavichord-mp3.js

Reverting back to dist/soundfont-player.min.js before this change works fine.

-Garrett

@danigb
Copy link
Owner

danigb commented Oct 5, 2016

Hi Garrett,

It's weird, the local examples works on my machine with the last changes... I'll take a look to it. Sorry for the inconvenience.

Saludos

@danigb danigb reopened this Oct 5, 2016
@danigb
Copy link
Owner

danigb commented Oct 6, 2016

Hi @grimmdude

I think I've fix the issue. Do you mind to test using the distribution file from master
Thanks

@grimmdude
Copy link
Author

Yes looks like that fixed it, thanks!

@danigb
Copy link
Owner

danigb commented Oct 6, 2016

Hi Garret, thanks for checking 👍

@danigb danigb closed this as completed Oct 6, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants