As @jkittner pointed out, items() returns a list of tuples of the form (key, value), so on the following line, an index of 4 will always error. The fix uses values() instead, so sat refers to a data structure that actually has a 5th element.
Future CI work will make sure that this will get caught by tests before merging (see discussion in PR#96 in this repo)
As @jkittner pointed out,
items()
returns a list of tuples of the form(key, value)
, so on the following line, an index of4
will always error. The fix usesvalues()
instead, sosat
refers to a data structure that actually has a 5th element.Future CI work will make sure that this will get caught by tests before merging (see discussion in PR#96 in this repo)