ykjit / yk

yk packages
https://ykjit.github.io/yk/
Other
28 stars 7 forks source link

More consistent `idx` naming #1256

Closed ltratt closed 1 month ago

ltratt commented 1 month ago

This PR renames a number of things surrounding index types. The main motivation was that we used (very) different conventions for iidx. Once I'd done those in the first few commits, it seemed sensible to plough on and make more things consistent with that convention.

There are still some other cases we could do (e.g. perhaps function names should be iter_iidxs and not iter_inst_idxs), but this PR probably removes 80-90% of the inconsistency, so waiting longer to get the last bits probably isn't worth our while right now.