Review comments

Author: 8BitJosh

programming/arduino/custom_firmware.md

@@ -17,10 +17,9 @@ By default when the `robot` object is created it will try to communicate with al
 If you want the API to not try connecting to a device we need to *ignore* the Arduino.
 
 To configure a `Robot` object to ignore a Arduino with custom firmware, you will need to provide it with the Arduino's serial number.
-The Arduino serial number is a character string of numbers and letters, and is output in the robot log when you run a program on your robot with your
-Arduino connected.
+The Arduino serial number is a string of numbers and letters, and is output in the robot log when you run a program on your robot with your Arduino connected.
 
-You'll need the ID later, so it's best to save it into a variable:
+You'll need the serial number later, so it's best to save it into a variable:
 
 ~~~~~ python
 from sr.robot3 import *
@@ -86,6 +85,7 @@ Putting a `b` in front of a string is a short hand way of creating a bytes objec
 
 ~~~~~ python
 # This will send the message "data" over the serial port
+# The b in front of the string is not a typo, this is a byte string
 robot.raw_serial_devices[ARDUINO_SN].write(b"data")
 ~~~~~
 

programming/arduino/index.md

@@ -7,6 +7,7 @@ title: Arduino API
 
 The Arduino that came as part of your kit was shipped with a firmware that provides the functionality outlined in the [Arduino API](/docs/programming/arduino/sr_firmware) page.
 You may wish to use the Arduino as is, extend the functionality of this firmware, or completely replace it.
+You can also add your own Arduinos, or other boards, to the kit, running either our firmware or your own.
 
 The `sr.robot3` library provides support for using the Arduino in three possible configurations:
 

programming/arduino/sr_firmware.md

@@ -77,34 +77,34 @@ robot.arduino.pins[4].mode = INPUT_PULLUP
 You can read a **digital** input pin with the following code:
 
 ~~~~~ python
-# robot.arduinos[ARDUINO_SERIAL_NUMBER].pins[PIN_NO].digital_read()
-
 # to read Arduino's digital pin 3...
-pin3 = robot.arduino.pins[3].digital_read()
+value = robot.arduino.pins[3].digital_read()
+
+# to read Arduino's digital pin 7...
+value = robot.arduino.pins[7].digital_read()
 ~~~~~
 
-`pin3` will now contain `True` or `False` depending on whether the pin was high (5v) or low (0v), respectively.
+`value` will now contain `True` or `False` depending on whether the pin was high (5v) or low (0v), respectively.
 
 You can read an **analog** input pin with the following code:
 
 ~~~~~ python
-# robot.arduinos[ARDUINO_SERIAL_NUMBER].pins[PIN_NO].analog_read()
-
 # to read Arduino's analog pin A0...
-pinA0 = robot.arduino.pins[A0].analog_read()
+value = robot.arduino.pins[A0].analog_read()
+
+# to read Arduino's analog pin A4...
+value = robot.arduino.pins[A4].analog_read()
 ~~~~~
 
 The analog pin numbers are available as `A0`, `A1`, `A2`, `A3`, `A4`, and `A5` respectively.
 
 
 ## Output
 
-You can only set digital outputs (there's no analog_write, although feel free to modify the Arduino's firmware to add the ability to output [PWM](https://wikipedia.org/wiki/Pulse-width_modulation "Pulse-width modulation") if you desire).
+You can only set digital outputs (there's no analog_write, although feel free to modify the Arduino's firmware to add the ability to output [PWM](https://wikipedia.org/wiki/Pulse-width_modulation) (Pulse-width modulation) if you desire).
 To set a digital output pin, you would use the following:
 
 ~~~~~ python
-# robot.arduinos[ARDUINO_SERIAL_NUMBER].pins[PIN_NO].digital_write(VALUE)
-
 # to set Arduino's pin 2 high:
 robot.arduino.pins[2].digital_write(True)