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

BB-48: General user-defined types (beyond trivially copyable) #23

Open
mcmillan03 opened this issue Sep 3, 2021 · 1 comment
Open

Comments

@mcmillan03
Copy link
Member

General user types: Look at the HPEC'18 futures paper. For a generalized user type, we need to add methods to copy, create, and delete edges. Our user defined types today require that the type work with a memcopy. Our current approach is limited. Aydin has experimented with this for years and has never come up with a workable solution. Is there even a need for this? Can’t we cover most cases by using an integer key as the values in the GrB_matrix with that key points into a vector of user defined objects. But this doesn’t let you use the values referenced by the key in the GraphBLAS operations. A good example of when something like this might be needed is when processing strings.

@DrTimothyAldenDavis
Copy link
Member

I've thought a lot about this too and don't have a good solution. RedisGraph creates property graphs without the need for variable-sized user-defined types. Adding variable-sized dynamic types to my library would be incredibly difficult for me to do. In my opinion this issue should be closed.

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

2 participants