google/wwwbasic

Implement PLAY and SOUND

flagxor opened this issue · 7 comments

We should implement PLAY + SOUND.

Sounds like thomasmcneill has this implement here:
https://github.com/thomasmcneill/wwwbasic/commits?author=thomasmcneill

It looks like your changes are actually on master of your branch, which sound be mergeable.
It should work to create another pull request against master.
You want it to look like this:
https://pasteboard.co/HFjPsKk.png

That should be the default if you click 'pull request' under the clone and download button from here:
https://github.com/thomasmcneill/wwwbasic

Hi Thomas,

Looking at your branch more closely, there's a few licensing issues we'd need to sort through before merge (to keep the lawyercats at bay :-). See below.
I'll happy to rework things as you've done the hard part of getting it running!
Unless you'd rather?
I've also figured out how to create a PR with your master changes and then override the CLA bot.

  • For the note frequencies, the license/copyright notice on the page seems to forbid copying (as opposed to linking): https://pages.mtu.edu/~suits/PofMCopyRight.html
    • Calculating the note frequencies can just be done with Math.pow(2, n/12) * 110 or some such, so we can likely substitute a formula.
  • For the PLAY docs, they fall under CC-3.0, https://creativecommons.org/licenses/by-sa/3.0/, that carries different limits than Apache, so it might be better to link to that rather than inline.
  • The sample with PLAY has something similar. We could put it in third_party, but that seems tedious. Alternatively, I'm sure I can come up with an unencumbered jingle for the example.

I'll likely get cycles to do this closer to the weekend.

Cheers

-BradN

Looking forward to sound implementation as well. Hope you work it out.

Example program (for now):
http://aleatoryrecords.com/