Daniel Beskin
Daniel Beskin
Do you suggest using `PathFunction` instead of axes or in addition to it?
What's nice about path functions is that they shouldn't require any changes to the zipper. The only complication I can think of, is that there is a class invariant regarding...
I still have reservations about the readability of the whole thing, feels like the `Zipper` is being spread too thin between the numerous traits. Also, before merging into master, the...
I can think of multiple possible bottlenecks in the code: - Freely hashing and sorting paths. - Using `ZipperHoleMap` for depth first iteration. As noted in the code it was...
Adding the operations discussed in #76. The same comments about performance still apply.
I think this was fixed in #67, can you try loading your files using the version on `master`?
Thanks for the quick response. My googling skills failed me... I tried searching for a similar issue but didn't find the one you linked to. In any case, I'm all...
Thanks for looking into this, let me know if there's anything I can do to help.
Wow... Out of curiosity, what sort of failures does this cause? I would've imagined that it will only trigger pattern-match exhaustiveness warnings and maybe some type-inference issues (unless reflection is...
Ah, that makes sense. Too bad if this won't go in. But seeing how you probably invested more time into this then I spent on my changes, I appreciate the...