UCL / MPHY0026

Other
18 stars 20 forks source link

Updates to Launch 2023 #64

Closed MattClarkson closed 1 year ago

tdowrick commented 1 year ago

Testing on Windows

17.4.1 Camera Calibration

17.4.2 Make your own pointer

17.4.3 FRED

Seems fine to me, web app so unlikely to be any issues since last time.

17.4.4 AR Overlay

All changes are currently on https://github.com/UCL/MPHY0026/tree/64-tom

@thompson318 any thoughts on the issues with camera calibration/pointer apps?

I can also give it a run through on my Ubuntu laptop.

tdowrick commented 1 year ago

Other thought - scikit-surgerpclcpp and opencvcpp are dependencies for MPHY0026 but as far as I can tell they aren't used anywhere.

If we can remove these it may make it easier to run with different versions of Python, as currently they only have wheels up to 3.7.

tdowrick commented 1 year ago

Ubuntu

Camera Calibraiton, Pointer, and FRED same as above.

AR Overlay not working, related to the issue with VTK windows on Ubuntu?

thompson318 commented 1 year ago

Other thought - scikit-surgerpclcpp and opencvcpp are ....

They were used in surgery-calibration, but should have been removed now, so should be removed from MPHY0026

thompson318 commented 1 year ago

Not currently working for me - no overlay showing on the Aruco marker. Will investigate further.

I suspect this is because the configuration file described in the docs is out of date. Did you copy and paste from the docs? It needs to updated like this https://github.com/UCL/MPHY0026/commit/b7de62bfe3dd215af6191cb4150163ea91091389, see issue #62. I can do that today or tomorrow.

thompson318 commented 1 year ago

AR Overlay not working, related to the issue with VTK windows on Ubuntu

I've created a version of BARD that works for me (ubuntu) with VTK > 9. It's here https://github.com/SciKit-Surgery/scikit-surgerybard/tree/101-skutils. It needs sksurgeryutils>1.2.1 and takes the Start and Initialize commands out of the constructor for vtk_overlay window. @tdowrick Can you test it on windows? @MattClarkson and mac?

thompson318 commented 1 year ago

I've fixed the pivot calibration documentation. Fix is here: https://github.com/UCL/MPHY0026/tree/62-fix-config. There's a quite a few pylint problems that should be resolved first (https://github.com/UCL/MPHY0026/actions)

tdowrick commented 1 year ago

I've fixed the pivot calibration documentation. Fix is here: https://github.com/UCL/MPHY0026/tree/62-fix-config. There's a quite a few pylint problems that should be resolved first (https://github.com/UCL/MPHY0026/actions)

Yes this works for me now.

tdowrick commented 1 year ago

I have fixed linting errors and merged mine and Steve's recent changes to master.

tdowrick commented 1 year ago

AR Overlay not working, related to the issue with VTK windows on Ubuntu

I've created a version of BARD that works for me (ubuntu) with VTK > 9. It's here https://github.com/SciKit-Surgery/scikit-surgerybard/tree/101-skutils. It needs sksurgeryutils>1.2.1 and takes the Start and Initialize commands out of the constructor for vtk_overlay window. @tdowrick Can you test it on windows? @MattClarkson and mac?

Works for me on Windows

thompson318 commented 1 year ago

I've updated BARD to use an up to date version of videocalibration videocalibrationchecker from sksutils. I think the spamming should be fixed now.

tdowrick commented 1 year ago

There were a few conflicting requirements in MPHY0026 and the new BARD, but I have resolved these now and the issues seems to be resolved now.

I don't think there are any more changes to make, so will close now.