tezpool icon indicating copy to clipboard operation
tezpool copied to clipboard

Published 20 hours ago • verified publisher icon dakk

getCycleSnapshot is incorrect

Open vogelito opened this issue 6 years ago • 2 comments

the getCycleSnapshot function currently uses the following to determine the block index of the snapshot at a given cycle:

https://github.com/dakk/tezpool/blob/847f855ff26246039bc1d477a99eaf3c7038a6a1/tezpool.py#L81 This is wrong. However, when looking into fixing it I noticed that just above that there's a commented bit that's a lot closer to what I'd expect would be correct: https://github.com/dakk/tezpool/blob/847f855ff26246039bc1d477a99eaf3c7038a6a1/tezpool.py#L76-L79

Before I started digging in more, I wanted to ask why are the bits commented out?

vogelito avatar Oct 06 '18 20:10 vogelito