Small bug in algos/base.py

[anniezhi]

Hi, I think line 127 and line 167 both use 'i' as the enumerating index, but they form a nested loop. This can cause incorrect index referencing once 'done' is not full of 'False'.

[lcswillems]

Hi Xiaoying, I don't see the bug you are referring too. The variable i is not used after the for loop line 167, so how could it cause a bug?

[anniezhi]

Hi Lucas, Thanks for getting back. Variable 'i' is still used in the for loop started at line 127, after exiting the for loop in line 167. The for loop at line 167 is nested within the for loop at line 127.

[anniezhi]

I see. Index variables with the same name in nested loops do not cause problems anymore :) In that case the current code works perfectly.