Skip to content
This repository has been archived by the owner on Nov 9, 2023. It is now read-only.

Nearest User Node #57

Closed
DoubleStyx opened this issue Aug 23, 2022 · 11 comments
Closed

Nearest User Node #57

DoubleStyx opened this issue Aug 23, 2022 · 11 comments
Labels
LogiX New Node Use this for when you want a new node added

Comments

@DoubleStyx
Copy link
Contributor

Like Nearest User Head, but gets the User ref instead.

@Xlinka Xlinka added New Node Use this for when you want a new node added LogiX labels Aug 23, 2022
@LeCloutPanda
Copy link
Contributor

Duplicate of #39 but looking into it anyways

@DoubleStyx
Copy link
Contributor Author

DoubleStyx commented Aug 23, 2022

We probably shouldn't have issues like #39 listing two unrelated nodes like that anyway. I think keeping this issue open is justifiable.

@reality-exe
Copy link

Yeah i should've just made a new issue for that.

@jeanahelver
Copy link
Contributor

sorry to add to this but would it be possible to get a list for ignore user similar to + , format or index of first match
finding the second nearest user is difficult in vanilla

@LeCloutPanda
Copy link
Contributor

It's possible just gotta look into it more, hopefully by the next release it's done

@Xlinka
Copy link
Owner

Xlinka commented Aug 25, 2022

is this a issue on the main github ?

@DoubleStyx
Copy link
Contributor Author

is this a issue on the main github ?

Not that I know of.

@Xlinka
Copy link
Owner

Xlinka commented Aug 25, 2022

Please make one so I can then link them.

@DoubleStyx
Copy link
Contributor Author

Hm, Nearest User Hand actually has an output for the User ref of the nearest user's hand. Now I'm confused. Did I just create an issue for something that already had a solution? Well anyway, I guess we could add nodes for other avatar points, like the hips, foot, chest, and so on. I'm not sure what the use case for such nodes would be, but it is an interesting proposition.

@reality-exe
Copy link

There's also one for Nearest User Head. However it would be nice to have just the user reference

@DoubleStyx
Copy link
Contributor Author

I don't think there's any harm in having a single node that provides multiple related outputs. There is a small performance cost involved in evaluating any outputs that aren't used, but many nodes already follow this output-batching design philosophy.

@Xlinka Xlinka closed this as completed Mar 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
LogiX New Node Use this for when you want a new node added
Projects
None yet
Development

No branches or pull requests

5 participants