Eljas Roellin

Nice catch - agree on all points regarding inconsistency, the causing sections & the solution with @jlause. Made the PR implementing the "half-pseudocode" and added tests which catch your described...

There are some tests failing, solving this together with maybe some more overhauling - opened draft PR with examples to raise some suggestions, especially with examples, of what our ideas...

> Yeah, smaller test datasets are also fine. We have issues with some distances failing on smaller test datasets so we had to sadly keep them a bit on the...

Thanks for getting back to this so quickly! Checked a bit more into this, bottom line is probably that `order` is _not required_ in our case/the example above. Us calling...

Not sure what could be fixed here, since this > In any case, seems like you can just omit `order=keys` to keep getting the same plot you would have otherwise?...

Hey, thanks for the request. To be able to reproduce and help, it is a big aid for us if you can supply a code sample that we can run:...

> To be able to reproduce and help, it is a big aid for us if you can supply a code sample that we can run: that is, with some...

Ah yes I see, thanks for the pointer - if I see that correctly, the missing imports persist there in the screenshot of the fist comment, and the unaltered use...

> What surprises me is that the PR checks didn't catch it. I'm not heavily routined with all effects and traps of using `.. literalinclude::`, so my first guess would...

Hey - it would be most helpful to post user questions in the [scverse forum](https://discourse.scverse.org/) - there, other users encountering the same question will be able to find a response...