search

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

Exclusions: SS studies being excluded from TS studies #228

Closed thenineteen closed 3 years ago

thenineteen commented 3 years ago

Some datapoints are labelled both SS and ET/CES(=TS). For SS queries, exclusions correctly filters all ET studies, but not all CES.

For TS queries, all SS are being filtered even those with TS overlaps, which needs a dropna(subset=) fix

thenineteen commented 3 years ago

CES exclusions fixed in 564cf829b60075d92a675e75e2f1837411da13f1 SS fixed in 2edc25982bf2afb9333c5fbcd3f4bd8a57371d4e

thenineteen commented 3 years ago

CES exclusions fixed in 564cf82 makes sense as it treats CES and ET the same, which are "topological studies" i.e. p(Semiology| GIF)

However, TS queries shouldn't include SS-TS overlaps, as these may include cough and mesial temporal lobe cases. These overlaps will then only be "seen" by the query in all-data queries.