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

Switch to JSON.NET for Unity #23

Open
MystikalPooka opened this issue Dec 4, 2018 · 1 comment
Open

Switch to JSON.NET for Unity #23

MystikalPooka opened this issue Dec 4, 2018 · 1 comment

Comments

@MystikalPooka
Copy link
Owner

Newtonsoft fails when compiling for IL2CPP in certain iOS builds. Swapping to JSON.,NET should be basically an in-place swap and should fix these issues outright.

@MystikalPooka
Copy link
Owner Author

Might also want to look into using a lighter-weight serialization option as most of the states from behaviors are now being stripped in favor of stateless streams. The only reason I had JSON.net is becasue default Unity serializer compeltely fails for trees with depth > 3, which is quite common with less simple behavior trees. We could explore other options to serialize the trees.

Another discussion to have is whether or not to stay with the current Asset database system (using a scriptableObject). If we do not use the scriptableObject it opens up infinite possibilities for storage solutions for these trees, but it makes it slightly harder to integrate into the editor inspectors and can make compiles between editor refreshes take much longer...

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

No branches or pull requests

1 participant