search

ccp4 / DUI

CCP4 DIALS User Interface
GNU General Public License v2.0
5 stars 3 forks source link

Allow multiple definitions for parameters where `.multiple = True` #103

Open ndevenish opened 5 years ago

ndevenish commented 5 years ago

Saw user try

scan_range=1-600,900-1600

This doesn't work, because it needs to be passed as

scan_range=1-600 scan_range=90-1600

This actually fails with a nonsensical error, so perhaps there is a problem with the parsing interface also inside dials

dagewa commented 3 years ago

Assume this refers to the integrate step. This is a general problem for all parameters where .multiple = True in the PHIL definition. A widget that allows duplication ("add another row") would be a reasonable solution.