Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Typos and stuff I thought I'd clean up while following Part I #2

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

r4dian
Copy link

@r4dian r4dian commented Jul 11, 2022

Decided to learn SC properly instead of just editing examples without full understanding, and found this book on gitbooks.

Some examples were broken, mostly by "" and '' gettign formatted as “” and ‘ ’ and so on.

r4dian added 16 commits July 5, 2022 12:51
Gitbook disxussion link has been subject to bitrot, changed it to github issues.
(and these single ones ‘ ’ with ')
This makes it easier to find & replace characters like the problem quotes.

Added a coupld of TODO notes where some of the text seems wrong, but I don't don't understand enough to fix it yet.
It didn't work before & this is more equivalant of the lists example it's contrasted with.
That's your external ip, but you want your machines ip on your network, not on the wider internet. (Unless you want to also get into port forwarding and router config here...)
127.0.0.1, the loopback address, will do for this example.

In case anyone asks:
On Windows, get it from `gip` or `Get-NetIPConfiguration` in Powershell
On macOS/Linux you can find it in ifconfig as the most cross-platform way for all distros.
If the sine goes from -1 to 1, then -1+2=1 to 1+2=3 times 220 is 220 to 660, not 440 to 660.
Added some Windows shortcuts, and some emphasis,
Fixed a typo,
Removed one example that was repeated twice exactly the same,
... etc.
Couple of typos in the identifiers of synthdefs.
linebreaks in "> quote boxes" were broken
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant