Skip to content

behavior tree rewrite #2143

New issue

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

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

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

behavior tree rewrite #2143

wants to merge 1 commit into from

Conversation

doomedraven
Copy link
Collaborator

No description provided.

@doomedraven doomedraven added the help wanted Extra attention is needed label May 29, 2024
@doomedraven
Copy link
Collaborator Author

To prevent recursive errors on json reports, for some malware spawns a lot of processes, and we embed children into parent process

@xiangchen96
Copy link
Contributor

This works, but changing the schema of the json has some implications:

I would prefer a conf that limits the depth of the tree with a sensible default or being able to disable it (users can rebuild the tree themselves from processes if needed).

@doomedraven
Copy link
Collaborator Author

well sometime progress requires breaking changes, this rarely happens, but would be nice to handle this properly without loosing details, but yes quick fix i guess will be configurable limit, but that limit should be the same as json limit, so is only on/off as is uniq field that breaks

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants