search

BreadchainCoop / breadchain

2 stars 1 forks source link

Vote recasting #66

Closed RonTuretzky closed 2 months ago

RonTuretzky commented 2 months ago

63

RonTuretzky commented 2 months ago

I'd personally like to see this refactored a bit so the "reset" logic of undoing the previous vote distributions is more grouped together. Right now I see 3 iterations over _points and I think it can be reduced to two.

With 9539abf Reduced to two loops (I think we already recognized that there must be at least 2 because the first loop is a pre-requisite for the 2nd)

Grouped all recast logic to L252-L259