Skip to content
This repository has been archived by the owner on Dec 22, 2022. It is now read-only.

Reworking the graph_t and graph::build interface. #14

Merged
merged 11 commits into from
Dec 7, 2020
Merged

Reworking the graph_t and graph::build interface. #14

merged 11 commits into from
Dec 7, 2020

Conversation

neoblizz
Copy link
Member

@neoblizz neoblizz commented Dec 5, 2020

I'll update the details of the rework once it is complete.

@neoblizz neoblizz marked this pull request as draft December 5, 2020 20:49
@neoblizz neoblizz marked this pull request as ready for review December 7, 2020 22:32
@neoblizz
Copy link
Member Author

neoblizz commented Dec 7, 2020

  • graph_container_t: removed
  • graph_base_t: removed
  • graph_t is not diamond inheritance
  • graph_t inherits csr, coo or csc
  • reworked build functions
  • polymorphic inheritance fix on CUDA removed
  • supports indices -> offsets and offsets -> indices conversions

@neoblizz neoblizz merged commit 805564c into master Dec 7, 2020
@neoblizz neoblizz deleted the graph_container_t branch December 8, 2020 18:48
@neoblizz neoblizz restored the graph_container_t branch December 8, 2020 18:48
@neoblizz neoblizz deleted the graph_container_t branch December 8, 2020 18:48
@neoblizz neoblizz added 🏡 api API related changes, suggestions, issues. 🍍 algorithms New or existing graph algorithms question. labels Jun 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
🍍 algorithms New or existing graph algorithms question. 🏡 api API related changes, suggestions, issues.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@neoblizz