radiology-research / ispy2_mri

GUI to enter info on incoming MRI's for ISPY2 into database
GNU General Public License v3.0
0 stars 0 forks source link

breastdb visibility issue #2

Closed jedchan closed 1 year ago

jedchan commented 1 year ago

Hi @RossBoylan

Is there a reason for this to be in github.com and not UCSF github? The breastdb mentioned in this repo does not exist anywhere in the public internet. This is internal to UCSF Radiology and only accessible to UCSF students, faculty, staff and its affiliates.

Are there plans to make the database and/or schema accessible to a public audience in the future? Otherwise this should be moved to our git.ucsf.edu ORG

RossBoylan commented 1 year ago

It's on the public github, albeit under radiology-research, so that local users who are not registered with UCSF github can access it more easily. As the last sentence of the introductory README says "If you are not in UCSF Radiology, it is very unlikely this package is of interest."

It also fits better with registering the package on PyPI, also publicly accessible.

There is no patient data in any of the files. And there are no plans (AFAIK) to make the ispy2 data publicly available.

Ross


From: jedchan @.***> Sent: Thursday, October 12, 2023 6:46 PM To: radiology-research/ispy2_mri Cc: Boylan, Ross; Mention Subject: [radiology-research/ispy2_mri] breastdb issue (Issue #2)

Hi @RossBoylan Is there a reason for this to be in github. com and not UCSF github?. the breastdb mentioned in this repo does not exist anywhere in the public internet. This is internal to UCSF Radiology. Are there plans to make the database ZjQcmQRYFpfptBannerStart This Message Is From an External Sender This message came from outside your organization.

ZjQcmQRYFpfptBannerEnd

Hi @RossBoylanhttps://urldefense.com/v3/__https://github.com/RossBoylan__;!!LQC6Cpwp!trR6ZVBgrAngKTShFsh2RWKndmF_lI9AxE3eG9RaLu9KzomW6X37mx6H-cHjnDTMGODDcfGbu2LQDFZVckMCEaW4x48$

Is there a reason for this to be in github.com and not UCSF github?. the breastdb mentioned in this repo does not exist anywhere in the public internet. This is internal to UCSF Radiology.

Are there plans to make the database accessible to a public audience in the future?

— Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/radiology-research/ispy2_mri/issues/2__;!!LQC6Cpwp!trR6ZVBgrAngKTShFsh2RWKndmF_lI9AxE3eG9RaLu9KzomW6X37mx6H-cHjnDTMGODDcfGbu2LQDFZVckMCjTuFZ74$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AATGAMTUR7D5QYDLRHRQBFLX7CMQZANCNFSM6AAAAAA56N6LFM__;!!LQC6Cpwp!trR6ZVBgrAngKTShFsh2RWKndmF_lI9AxE3eG9RaLu9KzomW6X37mx6H-cHjnDTMGODDcfGbu2LQDFZVckMCeZpxsQM$. You are receiving this because you were mentioned.Message ID: @.***>

jedchan commented 1 year ago

Thanks. I'm still trying to wrap my head around how this will work, but if the code is going to be added to pypi, then this makes sense for it to be public.