-
-
Notifications
You must be signed in to change notification settings - Fork 376
Move the logic of this check to the sensor class #422
Copy link
Copy link
Open
Labels
Difficulty: Newbie FriendlyAre you a new contributor looking for something to do? Try this :)Are you a new contributor looking for something to do? Try this :)Priority: LowLess important right nowLess important right nowType: DiscussionFurther information is requestedFurther information is requestedType: EnhancementAdds or improves a featureAdds or improves a feature
Metadata
Metadata
Assignees
Labels
Difficulty: Newbie FriendlyAre you a new contributor looking for something to do? Try this :)Are you a new contributor looking for something to do? Try this :)Priority: LowLess important right nowLess important right nowType: DiscussionFurther information is requestedFurther information is requestedType: EnhancementAdds or improves a featureAdds or improves a feature
The sensor class should tell if the config has been changed.
It would avoid the need for check
&& sensor->getSensorType() != SensorTypeID::Unknown && sensor->getSensorType() != SensorTypeID::Empty;The challenge is probably to get the acknowledged back to the sensor.
SlimeVR-Tracker-ESP/src/network/connection.cpp
Lines 516 to 522 in ee48341