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

niDigital_EnableMatchFailCombination is missing from nidigital API #1068

Open
sbethur opened this issue Oct 30, 2019 · 6 comments · May be fixed by #2076
Open

niDigital_EnableMatchFailCombination is missing from nidigital API #1068

sbethur opened this issue Oct 30, 2019 · 6 comments · May be fixed by #2076

Comments

@sbethur
Copy link
Contributor

sbethur commented Oct 30, 2019

This is a multi-session method like these, but it also accepts a NI-Sync session handle.

@marcoskirsch
Copy link
Member

Moved to unplanned until #981 is planned for fix.

@marcoskirsch
Copy link
Member

marcoskirsch commented Apr 14, 2022

NI-Digital now supports multi-instrument sessions. This is the recommended usage. So we won't add niDigital_EnableMatchFailCombination to nidigital anymore.

@ni-jfitzger
Copy link
Collaborator

ni-jfitzger commented Sep 18, 2023

NI-Digital now supports multi-instrument sessions which handles the NI-Sync driver internally. This is the recommended usage. So we won't add niDigital_EnableMatchFailCombination to nidigital anymore.

It's actually NI-TClk that's handled internally by the multi-instrument session. Whether using a multi-instrument session or multiple individual sessions NI-Sync will be called by the NI-Digital API, but opening and closing the NI-Sync session is still handled by the NI-Sync API.

Whether using a multi-instrument session or multiple individual sessions with NI-TClk, niDigital_EnableMatchFailCombination is still the entrypoint that a user would call to do whatever it does. There is no alternative entrypoint, so I'm reopening this.

We should maybe only support calling it from a multi-instrument session, though.

@ni-jfitzger ni-jfitzger reopened this Sep 18, 2023
@ni-jfitzger
Copy link
Collaborator

Note that this function still wouldn't be useful without a way to open an NI-Sync session, so it will remain unplanned, for now.

@doczhivago
Copy link
Contributor

@ni-jfitzger Here is a potential workaround for the NI-Sync session:

https://github.com/ni/miv-examples/tree/main/src/NI-SyncPython

@marcoskirsch
Copy link
Member

A pre-release version of an official nisync API was published.
https://pypi.org/project/nisync/

@ni-jfitzger ni-jfitzger linked a pull request Feb 2, 2025 that will close this issue
3 tasks
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.

5 participants