thenineteen / Semiology-Visualisation-Tool

Data driven 3D brain visualisation of semiology. Semiology to anatomy translator based on over 4600 patients from 309 peer-reviewed articles.
MIT License
9 stars 6 forks source link

when integrating lateralising only data (lat_not_loc), use subset of hemispheric GIFs already in all_gifs_combined #171

Closed thenineteen closed 3 years ago

thenineteen commented 4 years ago

Only pd.concat(join=inner) (if reindexed) or Gif Parcellations that already exist in all_combined_gifs

this avoids excess addition of false localising values to irrelevant regions

related to the changes made in pull #167

thenineteen commented 4 years ago

there is no right way about this

this is the idea that we are integrating lateralisation and localisation data rather than querying them separately. And if we already have some localisation data even from other patients and publications, then it may be easier on the eye to see few brain regions light up and more accurate. OTherwise, the lateralisation which exceed localisation or the lateralisations without localisations result in entire hemispheres lighting up which can give the false impression of data suggesting e.g. occipital lobe resections resulting in seizure freedom from gelastic seizures.

thenineteen commented 4 years ago

This would have to be implemented for both lat_but_not_loc and lat_exceeds_loc both in Q_L

thenineteen commented 4 years ago

given the current SVT (3D slicer GUI) has a Visualisation tab with Min. 2D opacity setting set at 0.25 (makes the top 25% opaque) - I think I will leave this feature out for now, because:

thenineteen commented 3 years ago

this is implemented better in the global lateralisation option