Because the nodes add +1 to each index, to turn a zero-based offset to a one-based offset, it makes negative indexes rather annoying:
- 1 = First image
- 2 = Second image
- 0 = Last image (instead of -1)
- -1 = Second-to-last image
If you want to keep the one-based offsets for positive numbers, I think special handling should be added to make it look more logical:
- 2 = Second image
- 1 = First image
- 0 = (error)
- -1 = Last image
- -2 = Second-to-last image
In other words, add an error for 0 and stop adding +1 when it sees negative numbers.
I'm not sure how best to make this 100% backwards-compatible, or at least allow people to notice the change in behavior. At least the zero error should catch most use cases. So, it might be best to just make the change, and note the behavior change in the README.
Because the nodes add +1 to each index, to turn a zero-based offset to a one-based offset, it makes negative indexes rather annoying:
If you want to keep the one-based offsets for positive numbers, I think special handling should be added to make it look more logical:
In other words, add an error for 0 and stop adding +1 when it sees negative numbers.
I'm not sure how best to make this 100% backwards-compatible, or at least allow people to notice the change in behavior. At least the zero error should catch most use cases. So, it might be best to just make the change, and note the behavior change in the README.