UCL / SkullBaseNavigation

Other
2 stars 1 forks source link

Load CT model from file, rather than retrieving it over OpenIGTLink from the StealthStation #15

Open tdowrick opened 3 years ago

tdowrick commented 3 years ago

In GitLab by @ThomasDowrick on Nov 23, 2018, 14:41

The current experimental setup retrieves the CT model from the StealthStation, using the OpenIGTLink Remote module.

In practice, we will be pre-loading the CT model to be used on the laptop, so the software should be able to load a local model instead.

tdowrick commented 3 years ago

In GitLab by @ThomasDowrick on Nov 23, 2018, 15:01

created branch 15-load-ct-model-from-file-rather-than-retrieving-it-over-openigtlink-from-the-stealthstation

tdowrick commented 3 years ago

In GitLab by @ThomasDowrick on Nov 23, 2018, 15:07

There are different methods of loading data, depending if it is DICOM or not.

@JonathanShapey - do you know what format the preoperative CT data will be stored in?

tdowrick commented 3 years ago

In GitLab by @ThomasDowrick on Nov 23, 2018, 15:45

If it isn't a DICOM file, than the following can be used:

slicer.util.loadVolume(file, properties={'name': volume_name})

The 'name' property is optional (by default the volume name is the filename), but as we will need to modify/access the loaded volume it is better to define the name.

A volume can also be loaded using slicer.util.openAddVolumeDialog(), but this doesn't allow for the name of the volume to be specified.

In terms of how it is implemented, there could either be a button to select/load the model, or the path to the model location could be specified in a configuration file and loaded automatically when the program starts. I have implemented a basic button/function to do this in https://weisslab.cs.ucl.ac.uk/WEISS/SoftwareRepositories/skullbasenavigation/commit/4913b68aba87f4f7de6088f6660b57fbb60cf40e

tdowrick commented 3 years ago

In GitLab by @ThomasDowrick on Nov 23, 2018, 16:16

mentioned in commit 4913b68aba87f4f7de6088f6660b57fbb60cf40e

tdowrick commented 3 years ago

In GitLab by @ThomasDowrick on Nov 23, 2018, 17:31

mentioned in commit 0d5a12851ebb02de2691f4acb63ceb85eaa172a8

tdowrick commented 3 years ago

In GitLab by @JonathanShapey on Dec 14, 2018, 13:53

I think that it would be best to have the capability to do both (i.e pull from StealthLink and load from file).

tdowrick commented 3 years ago

In GitLab by @JonathanShapey on Dec 14, 2018, 16:52

As discussed today, I think it would be beneficial to be able to load directly from Stealth as we will always register the patient (model in the lab) at the beginning of every case

tdowrick commented 3 years ago

In GitLab by @DavidPerez-Suarez on Feb 4, 2019, 18:06

mentioned in commit c4f76428aaaf1d42e2865d2cd266b2504e59804f

tdowrick commented 3 years ago

In GitLab by @DavidPerez-Suarez on Feb 4, 2019, 18:06

mentioned in commit f9ec6bb5725f72904f405b012e8efee754875a45

tdowrick commented 3 years ago

In GitLab by @RolandGuichard on Jun 24, 2019, 10:20

@ThomasDowrick and @AnastasisGeorgoulas I guess this issue can be closed ?

tdowrick commented 3 years ago

In GitLab by @AnastasisGeorgoulas on Jun 24, 2019, 11:52

If I remember right, we said we'd leave it so that we can discuss whether we should add a button for this or assume a default local location to check first, or something else.

tdowrick commented 3 years ago

In GitLab by @RolandGuichard on Jun 24, 2019, 13:13

hm, OK.

tdowrick commented 3 years ago

In GitLab by @ThomasDowrick on Jun 25, 2019, 09:10

Yes, I was going to double check what (if any) the requirement will be.

tdowrick commented 3 years ago

In GitLab by @JonathanShapey on Jun 25, 2019, 15:49

I think a button to check that all relevant images are loaded would be helpful. In patients, we will load

  1. MRI
  2. CT (most but not necessarily every time)
  3. Tumour volume
  4. Tractography volume

If there isn't time to add this before the demo, maybe we can discuss this on Thursday?