fli-iam / shanoir-ng

Shanoir (SHAring iN vivO Imaging Resources)
GNU General Public License v3.0
21 stars 21 forks source link

Manage timepoints and selection during import #2154

Open michaelkain opened 6 months ago

michaelkain commented 6 months ago

Hi @LaurentPV , I hope you are doing fine. I have been thinking after our discussion of this feature, that existed once in shanoir-old. We could add a tab "timepoint" to the study details, where the study admin can create a list of timepoints, e.g. with a text label, like V0,V1,V2 etc. When timepoints exist on the study level, the importing user has to select them during the import to tag each exam with an explicit time point flag. Maybe we could use the study/subject tag mechanism for that. The only change might be, that this selection, in relation to subject-tags are mandatory during the import. With kind regards, Michael

michaelkain commented 6 months ago

maybe we could add a mechanism, that we auto-tag depending on the time order of the imports, by patient, but this might be risky

LaurentPV commented 5 months ago

Indeed we could add an examination tag like the subject tag but I doubt that it would be used. For now Shanoir allows the use of the Examination note field to store which follow-up examination it is for the patient.

I thought that it would bring more confusion to the use of Shanoir Uploader that should be kept simple to use but with several discussions over the subject of follow up examinations and the evolution of a clinical file I think that an examination tag would be useful, especially if we add it to the solR engine.

michaelkain commented 4 months ago

Hi @LaurentPV, I think the above would be an interesting extension to work on. I agree, that today often the examination comment is used during the import to name it M0, M1, M2 or T0, T1, T2 etc. But the idea is, to oblige the user to do it on an exam, from a predefined list on the study level. This extension would have to be implemented in the webGUI + in ShUp and can be done completely independent. With kind regards, Michael

michaelkain commented 4 months ago

Maybe it would even be better to separate it from the tags and propose a slider on the study details level with the amount of time points possible for this study (no free label) and force the importing user to select a timepoint, e.g. propose the first timepoint available for this patient and just have the user validate it. With kind regards, Michael

michaelkain commented 4 months ago

and extend Solr as well for the timepoint filtering

LaurentPV commented 3 months ago

After talking about this topic with Solène from CHU Nord Nantes, it seems interesting for the user to have a timepoints tag as you describe it. Solene says that it should be on the exam level as well as the latest value on the subject level (priority for me on the exam level and maybe adding the last value on subject afterwards). And also that it should be added to SolR and mandatory during the import.