LVMakerHub / LINX

LabVIEW Community Edition LINX
Other
111 stars 114 forks source link

Read Pulse Width Simuluse Pulse Default Wrong #100

Open Nathan--B opened 8 years ago

Nathan--B commented 8 years ago

The default value for the "Stimulus Pulse" input on the "Digital Read Pulse Width" VI is 'None' but the documentation says that it is "Active High". This breaks the Ultrasonic Read.vi since it relies on the default values for both the Incoming Pulse and Stimulus Pulse.

However, I feel like the default for the Stimulus Pulse should be 'None' since "reading a pulse width" inherently requires no stimulus pulse (effectively a wrapper for pulseIn()). The stimulus pulse is an augmentation of the basic functionality. This then would require the Ultrasonic Read.vi to use constants for these inputs.

jagannath-sahoo commented 7 years ago

Please fix the bug as fast as possible