Skip to content
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

marker, or start pulse width #1873

Closed
okz opened this issue Dec 4, 2022 · 2 comments · Fixed by #2051
Closed

marker, or start pulse width #1873

okz opened this issue Dec 4, 2022 · 2 comments · Fixed by #2051

Comments

@okz
Copy link

okz commented Dec 4, 2022

Hi all,

Access to NIFGEN_ATTR_MARKER_EVENT_PULSE_WIDTH has been removed with issue #859 and pull #868.

This is disappointing, since at high sampling speeds, the pulse can be as short as ~150ns and can be missed by other 'sampling' HW.

Any chance of getting it back ? I am not aware of any other output, which is slower or can be configured slower than the sampling speed.

Kind regards,

@marcoskirsch
Copy link
Member

Hello @okz
Can you list the NI-FGEN attributes that you need?
There are ways you can workaround in the interim until we add them back, if you haven't.

a) You can create and add attribute members to your Session object.
b) You can call _set_attribute_vi_boolean, etc. private methods to set the properties directly.

@okz
Copy link
Author

okz commented Dec 15, 2022

Thank you, Ended up patching the attributes

Would suggest the group below, should be in. NI has documented this usecase specifically:

The Marker event pulse width, tm2, is at least 150 ns and can be significantly longer than 150 ns for slower Sample clocks. You can configure the pulse width by setting the Marker Event Pulse Width property or the NIFGEN_ATTR_MARKER_EVENT_PULSE_WIDTH attribute. Instruments commonly have a minimum pulse width specification for a trigger to be registered, and trigger pulses of smaller width are ignored. The signal generator ensures that a minimum pulse width exists on the Marker event by using a pulse stretching circuit. A Sample clock rate of 100 MS/s has a period of 10 ns, requiring the pulse to be lengthened for many devices to register the marker as a good trigger pulse. Refer to the device specifications for the timing specifications.

    * `started_event_pulse_width`
    * `done_event_pulse_width` 
    * `marker_event_pulse_width`
    * `started_event_pulse_width_units`
    * `done_event_pulse_width_units` 
    * `marker_event_pulse_width_units`

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

Successfully merging a pull request may close this issue.

3 participants