Page MenuHomePhabricator

clean up NDIConfigurationWidget
Closed, ResolvedPublic

Description

To make the widget a bit user friendlier, it would be nice to add some help texts in the UI and further show only buttons that actually make sense.

e.g. "add passive tool" + "discover wired tool" should only be shown if the corresponding tracking device has been connected.

Event Timeline

[SVN revision 23033]
FIX (#4047): some UI clean up; show/hide buttons according to connected tracking device

[SVN revision 23034]
FIX (#4047): remove not existing methods

Just a small remark: There are hybrid Polaris systems that use wired tools. These tools probably have their rom file 'onboard' and can be discovered using the NDI API, just like the wired Aurora tools. With the current code, the discover-wired-tools button is hidden for all Polaris systems, not ust the passive systems.

Do we have such hybrid tools for testing purpose?
I was wondering if this will work with one implementation, I mean do we need a further button or can this be realized with the same method "bool mitk::NDITrackingDevice::DiscoverWiredTools()" used for the Aurora system?!

We don't have the necessary hardware to test it. It should theoretically work with the same code. After all, it is not "Aurora" code, just "NDI" code and the API is the same for both systems. We don't need to put much effort into this, but we should keep it in mind for the future or for external users of MITK-IGT.