reset_iterator starts from the first validatorOrder value #114 #120

[RafaelVillar]

Hello @mottosso

Here is the PR for issue #120

[mottosso]

I've had a look at this and noticed a flaw in the design of the reset function. It's based on misunderstanding how the orders are interpreted, and how they represent not a starting point, but a range of values, and how that range is offset by 0.5. Tough one indeed. :(

I've added a comment to the offending line to help future readers. I did however write this on GitHub and did not test it. Could you let me know if this solves the issue for you @RafaelVillar?

[RafaelVillar]

Hey @mottosso I gave the edits you made a test and it seems to have worked as expected. It Fixes the issue it seems.

Let me know when you roll it into master

[mottosso]

@RafaelVillar All done, should be released as well as 0.8.12.