Skip to content

Improved Python Support for Aurora wired tools.#18

Merged
adamrankin merged 1 commit intoPlusToolkit:masterfrom
SciKit-Surgery:python-support-for-getTX
Feb 28, 2019
Merged

Improved Python Support for Aurora wired tools.#18
adamrankin merged 1 commit intoPlusToolkit:masterfrom
SciKit-Surgery:python-support-for-getTX

Conversation

@thompson318
Copy link
Copy Markdown
Contributor

Added ndiTX, ndiGetTXTransform, ndiGetTXPortStatus, ndiGetTXSystemStatus, ndiGetTXFrame, ndiGetPHSRNumberOfHandles, and ndiGetPHSRHandle to Python module

Added ndiTX, ndiGetTXTransform, ndiGetTXPortStatus, ndiGetTXSystemStatus, ndiGetTXFrame, ndiGetPHSRNumberOfHandles, and ndiGetPHSRHandle to Python module
Comment thread ndicapimodule.cxx
&_ndiConverter, &pol, &port))
{
result = ndiGetGXPortStatus(pol, port);
result = ndiGetBXPortStatus(pol, port);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not familiar with the GX calls, but is this correct?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure. I haven't used Py_ndiGetBXPortStatus in my application. I noticed when I was doing a find and replace that Py_ndiGetBXPortStatus called ndiGetGXPortStatus, which looked like a typo to me. We should check with @eruffaldi and I'll test it on one of my trackers.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just checked using our Polaris Vega. I think it was a bug, i.e. Py_ndiGetBXPortStatus always returns zero for me without this change, with this change I get an appropriate return value (49).

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sigh Sorry, that's my fault. Github diff shrunk the code between ndiGetGX and ndiGetBX so I thought you were changing GX to BX... disregard all of this.

@adamrankin adamrankin merged commit 5d537bb into PlusToolkit:master Feb 28, 2019
@thompson318 thompson318 deleted the python-support-for-getTX branch February 28, 2019 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants