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

Add New Sites #12

Open RossBoylan opened 11 months ago

RossBoylan commented 11 months ago

Background

ispy2 has many participating sites. From time to time new ones are added; I believe some drop out as well.

Since the current (v0.0.7) version of this tool and the web application do not permit addition of new site, the practice has been to leave this field blank and put the site in the comment.

Both applications use the database procedure select_ispy2_sites to get the list of sites.

Sites have both a number and a name; the widget shows both, sorting on the number.

Desired

A way to enter new sites from the GUI.

Possibilities:

  1. Add a + button next to the widget. When it is clicked, ask for number and name.
  2. Permit text entry in the field using a format like NNNN | name of site. This approach might make it too easy to create a site inadvertently, perhaps even when reading from a file.

In any case, the new entry should not be allowed to duplicate existing data.

Uncertainty

It is possible that creating a new site should entail more than adding entries to ispy2_sites, the only table referenced by select_ispy2_sites. So simply adding rows there might create some other problems.

Priority

We explicitly put this in the "do not delay delivery for this" bucket. Nice to have, not essential.