Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Aadded max wait time option to the time_limiter node #318

Closed
wants to merge 1 commit into from

Conversation

alextkd2003
Copy link

Description

There were no problems, but I think it would be useful when there are several instances of the same npc using this node, since we can get some random behavior.

Addressed issues

N/A

Screenshots

time limiter max wait time

@bitbrain
Copy link
Owner

Can you elaborate further what the issue is you are trying to solve with this?

Also, what exactly is the difference between wait time and max wait time?

@alextkd2003
Copy link
Author

hi @bitbrain this is not an issue is a nice to have feature. I constantly use the time_limiter node. I modified mine just to have that feature.

Let's say one npc is in idle mode and I'm using the time_limiter to set it to stay idle for 5 seconds and then go for a walk. If all my copies have the same value then they will all walk at the same time. If I change the values for copies, there will always be a pattern. But if the time is random, then I will have less predictable behavior since the states will be changing. In this way and all the copies of the npc will be doing different things.

I will add the same feature to all nodes with a timer to be honest, this is fantastic!! imo. I used in the pass Behavior Designer for Unity and I found that feature really cool, again imo.

Tree Ex.
image

@alextkd2003
Copy link
Author

About the differences about wait_time and max_wait_time. If max_wait_time is specified wait_time is the min_wait_time, if not specified and remains at 0 it works as it has always been working.

It may not be the best way to call the variables, but I wanted to change very little. I'll let you evaluate if these is a good idea, if not, nothing happens, in the same way all the scripts can be edited so the user can make this change if need it.

Thank you for all the effort you have put into this addon, of all the BTs I have tried there is no doubt that this is the best.

@bitbrain
Copy link
Owner

After much consideration, this is likely not going into Beehave V2, given its very specific and custom use case. However, I still think this is an interesting feature so feel free to raise a discussion here.

@bitbrain bitbrain closed this May 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants