mattytrentini / micropython_workshop Goto Github PK

It would be easy to think that if.isconnected() will block until the network is connected, instead of the actual functionality of immediately returning with the current network status. This should be clarified, and potentially the necessary wait loop provided:
while not sta_if.isconnected(): pass

It should be noted that the board should be unplugged prior to connecting or disconnecting a shield. While it can be done while powered, it greatly increases the chances of something going wrong and getting damaged - far better to just unplug the USB, take your time ensuring the shield orientation is correct and the shield is sufficiently pushed in, before reconnecting.

All files that are to be loaded onto the device prior to the workshop should be available in this repository. This will likely mean submoduling in any necessary external files, such as the LED Matrix driver and the OLED display driver.

It's easy to think that when you set a pixel, it should update (which is not the case). This could be clarified by putting a comment in the code example with the np.write() line.

Either fill it out or delete it, something must be done.

In the LED Basics section (of the getting started page) it is expected that the LED is inverted (like the Wemos D1 Mini one is). This isn't the case on the TTGO ESP32 board, and so this should be rectified.

The random / urandom documentation has seemingly no documentation in the official MicroPython documentation, and the functions available in ESP8266 vs ESP32 are different. ESP32 should be explicitly targeted most likely as it has more useful functions, and these functions should be specified better.

random is mentioned in the Digital Dice RGB LED Shield exercise.

Probably fork micropython-lib and add it there?

On the ESP32 board, the buzzer shield will only make noise once before having to be reset to do it again. The cause of this should be identified to ease the user experience when developing / testing.

In order for communications to work on Linux, the tty must have the correct permissions set. This should be added to the Linux install instructions.

• PIR
• Button
• Ambient light sensor
• IR controller

The d1_mini (dee one) import could be confused for dl_mini (dee ell) - a clarifying statement could help reduce this confusion.

Mu looks more appropriate for beginners and ought to work consistently across the major platforms.
Note: Currently the beta is required for ESP8266/32 comms.

• Update images
• Make first example a bit prettier