Skip to content

Read Pulse Width Simuluse Pulse Default Wrong #100

Description

@Nathan--B

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.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions