FNNDSC / ChRIS_ultron_backEnd

Backend for ChRIS
https://fnndsc.github.io/ChRIS_ultron_backEnd
MIT License
31 stars 100 forks source link

Allow for non-binary PatientSex #433

Open jennydaman opened 2 years ago

jennydaman commented 2 years ago

https://github.com/FNNDSC/ChRIS_ultron_backEnd/blob/f1804e4a6eead9e043674769575c3494ff1be094/chris_backend/pacsfiles/models.py#L23

In some cases e.g. anonymized data, PatientSex might be specified as "O" e.g. consider this series: https://www.visus.com/fileadmin/content/pictures/Downloads/JiveX_DICOME_Viewer/case1.zip

It is necessary to allow for PatientSex to be of values other than just "M" or "F"

cornelia247 commented 2 years ago

hello, I am Cornelia, an Outreachy Applicant, I am interested in this issue, please Can it be assigned to me?

Motunrayo321 commented 2 years ago

Hello @jennydaman, My name is Motunrayo. I'm an outreachy applicany. Can this issue still be assigned to me as well? I would like to work on it.

jennydaman commented 2 years ago

@Motunrayo321 @cornelia247 has already successfully implemented the fix in #440, we are just waiting for finishing touches.

tr-sangale commented 1 year ago

@jennydaman i hope that this issue can be closed