Skip to content

Correction on getResultIDForParamCombination

The current implementation of the method getResultIDForParamCombination of the parameter class presents a behaviour that can be prone to errors.

The loop that goes down over the resulIDsList is defined like this:

for resultID in resultIDsList[1:]:

This has the effect of ignoring the first element of the list. I am uncertain on whether this has been done. Probably to skip runID=0. Nevertheless, I have realized that this can lead to trouble because under certain circumstances CST will return a resultIDsList that will begin with runID=1, and in this case we won't be interested in skipping the first element of the list.

Therefore, I think that this loop should be modified to avoid skipping the first resultID of the list.