some more TODO things
update docstring on neighbours
we shouldn't be caching this