T
tscircuitβ€’3w ago
Seve

Seve - @i use arch btw. nice fix on the math-ut...

@i use arch btw. nice fix on the math-utils, merged it and doing the rename/deprecation of the old fn. Something I wanted to mention: You can use yalc to publish to a "local registry" for testing modules so you don't have to wait to merge. thanks for the contrib 😁
4 Replies
itsme
itsmeβ€’3w ago
Seve
SeveOPβ€’3w ago
Looks great, nice fix. Kind of crazy I want to make sure drc gets fixed, i can help if youd like. Are you also bounty hunting? Dont forget β€œ/claim” if you are, optional ofc Im still reasoning around how this caused the failure, something seems off to me, i think manhattan distance is appropriate in that case It seems like the actual issue was the trace distance not being added Manhattan distance is more relevant for spatial indexing usually
itsme
itsmeβ€’3w ago
The Manhattan distance was higher than the actual clearance so the obstacle was filtered out with the assumption that it was far enough away.
Seve
SeveOPβ€’3w ago
Hmm ok cant hurt

Did you find this page helpful?