d_min on integrate: Clicking up then down fails to run

ccp4 / DUI

CCP4 DIALS User Interface

GNU General Public License v2.0

5 stars 3 forks source link

d_min on integrate: Clicking up then down fails to run #143

Closed ndevenish closed 4 years ago

ndevenish commented 4 years ago

Dui 2012.1: On integrate simple page. d_min says None: Click up: Click down: Click run. Integrate fails because 0 gets passed through as d_min.

prediction {
  d_min = 0
}
...
Sorry: Prediction at d_min of 0.0 is not possible with wavelength 1.23985

dagewa commented 4 years ago

I've been looking at this. I can see why it doesn't work and how to catch the case where we'd like to reset the d_min parameter. However I can't see any way to actually remove prediction.d_min from the parameters after it has been set once. @luisodls how are you supposed to reset a single parameter from within the IntegrateSimplerParamTab class?

dagewa commented 4 years ago

Found it, but see commit message.