-
Notifications
You must be signed in to change notification settings - Fork 19
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
Feature/add medium linear motor #105
Feature/add medium linear motor #105
Conversation
@@ -15,7 +15,7 @@ public enum DeviceType : ushort | |||
ExternalTiltSensor = 0x0022, // TILT_SENSOR | |||
MotionSensor = 0x0023, // MOTION_SENSOR | |||
VisionSensor = 0x0025, // COLOR_DISTANCE_SENSOR | |||
ExternalMotorWithTacho = 0x0026, // MEDIUM_LINEAR_MOTOR | |||
MediumLinearMotor = 0x0026, // MEDIUM_LINEAR_MOTOR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Naming choice is perfect. I copied these from poweredup-node ... but they have history.
Normally, this would apply for a breaking change. But honestly, the motor is not implemented so I cannot imagine why someone would use this enum value.
Therefore, fine with me!
|
||
namespace Example | ||
{ | ||
public class ExampleTwoPortHubMediumLinearMotor : BaseExample |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I usually add the test run to the Program.cs commented out ... like that it can be easier discovered. Fix it on the next PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, good idea! Will take care of that in the next PR
Closes #102 |
I have tested this one with the TwoPort Hub and the MediumLinear Motor. I have changed the naming of the DeviceType from ExternalMotorWithTacho to MediumLinearMotor according to the Lego naming. Hope that is ok.
Closes #102