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

TreeNode data dict #1249

Open
Licini opened this issue Jan 5, 2024 · 0 comments
Open

TreeNode data dict #1249

Licini opened this issue Jan 5, 2024 · 0 comments
Assignees

Comments

@Licini
Copy link
Contributor

Licini commented Jan 5, 2024

          for the tree node there are two options
  1. keep the attributes dict in the data dict and convert the node explicitly to/from data
  2. don't include the attributes dict in the data dict and let the encoder/decoder handle to/from (then the attributes are included implicitly)

the file size of 1. is much smaller and is similar to what happens to nodes in a graph and vertices in a mesh. however, there the behaviour is not so explicit since the nodes and vertices are not separate objects.

if a tree node is not meant to exist on its own (like graph nodes and mesh vertices), perhaps we should make sure that they cannot be converted to/from data independently?

Originally posted by @tomvanmele in #1247 (comment)

@Licini Licini self-assigned this Jan 5, 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

No branches or pull requests

1 participant