arduino / eduintro Goto Github PK

Is there a reason you use sequential if statements, instead of else if? E.g. in

https://github.com/arduino/EduIntro/blob/master/examples/by_topic/PhysicalPixel/PhysicalPixel.ino#L44

Kind regards,
Thomas

int ServoMotor::write(uint8_t _value)
{
  if (!_attached) {
    attach(pin);
    _attached = true;
    }
  Servo::write(_value);
}

Servo:: Write does not return a value, so should it also be set to void here?

Hi, I really like that only a single include is needed:

#include <EduIntro.h>

Everything else being so neat and simple, how about:

#include <Basics.h>

Just like the File > Example > Basics > ...

Or something else a little friendlier to beginners.

(I understand you named it for a specific workshop.)

Kind regards,
Thomas

hi
it seems that servo.h and virtualWire.h both are using same timer. so when using with virtualWire compile error will occur.
im using uno and the error message is:

libraries\Servo\avr\Servo.cpp.o (symbol from plugin): In function `ServoCount':

(.text+0x0): multiple definition of `__vector_6'

libraries\VirtualWire\VirtualWire.cpp.o (symbol from plugin):(.text+0x0): first defined here

Hi,

Would it make sense to introduce something like button.pressed() instead of readSwitch() ?
E.g. in https://github.com/arduino/EduIntro/blob/master/examples/by_topic/Button/Button.ino#L29

if (button.readSwitch() == LOW) { led.on(); } else { led.off(); }

if (button.pressed()) { led.on(); } else { led.off(); }

Or isPressed(), as in isPWM()

Kind regards,
Thomas

Hey. I got an idea as to include function for morse code which imports functions for basic applications like sending a distress sos signal and I'm working on it. Not just the sos signal, we can include the functions for every alphabet in morse code and help in communicating in the morse code itself. The small and long intervals can be represented by change in duration and pause in blinking of the LED connected.
May I know whether anyone's working on this issue ? I'd very much like to contribute in this regard