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

Feature/badbehaviour #333

Draft
wants to merge 9 commits into
base: Preview4_0
Choose a base branch
from

Conversation

lukaspj
Copy link
Contributor

@lukaspj lukaspj commented Oct 3, 2020

Add bad behaviour system

Note: I tried updating this to the module system but I haven't properly tested this, this should be seen as a PoC'

Original PR from the GarageGames repo:
GarageGames/Torque3D#2120

@lukaspj lukaspj added Art Issue Broken images/models, Possible licence conflicts enhancement New feature or request help wanted Extra attention is needed Script Issue as part of template/module script labels Oct 3, 2020
@lukaspj
Copy link
Contributor Author

lukaspj commented Oct 3, 2020

Seems to work, but needs UI updates to play nicely with dark-theme and could use proper testing.

@ghost
Copy link

ghost commented Oct 8, 2020

BadBehavior itself works fine and is easy to integrate (just copy the folder), I'm using it for years, the only problem is, that the UI does not work, so I created my behavior trees using my code editor instead of the UI inside the engine, this is the reason I hesitated to make a pull request of it into the main repo.

The issue with BadBehavior see here: BadBehavior/BadBehavior_T3D#2

I would just add it to the engine and have others fix the problem, I mean it is not the first unusable feature that was added to the engine without ever testing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Art Issue Broken images/models, Possible licence conflicts enhancement New feature or request help wanted Extra attention is needed Script Issue as part of template/module script
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants