-
Notifications
You must be signed in to change notification settings - Fork 39
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
Updating the JS binding for recent changes #118
Comments
Here are the changes, I hope did not miss anything. Btw, thanks for opening this, it will also be useful for C++ bindings. Motors
Sensors
Buttons, RemoteControl
Leds This is the biggest change in specification.
# Set left logical led to red color.
Leds.set_color(Leds.LEFT, Leds.RED)
# Set trigger for the right logical led, make it 50% bright:
Leds.set(Leds.RIGHT, trigger='timer', brightness_pct=0.5) This also scales well onto other platforms. For example, PiStorms have two logical groups, each having Red, Blue, and Green physical leds.
Sound
The way we currently work with different platforms may change in the future, see discussion in ev3dev/ev3dev#412. Also see #115. |
Great! Thanks. I'll start working on these changes today. |
@ddemidov I noticed that the |
Oh, and what's the current status on detecting the running platform for buttons/LEDs? I saw a long discussion somewhere referencing a fair amount of Linux magic that I didn't understand... was that concluded? If not, what are you doing for C++ and Python in this area? |
I don't see why not.
I think that is still in the planning phase. I made ev3dev/ev3dev-lang-python#90 with this in mind, but nothing besides that. |
@ddemidov Can I get a list of the changes that I need to adapt to on my Node.js side? I believe that there were changes to the LED spec, but didn't follow closely enough to fully grasp the changes that I should make. Also, what's the situation regarding supporting the various ev3dev platforms (RasPi, BB, etc)? Do we need to change the default instances?
P.S. Sorry I've been so out-of-touch 😉
UPDATE: Current TODO list
The text was updated successfully, but these errors were encountered: