-
Notifications
You must be signed in to change notification settings - Fork 268
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
Adding support for graph.AdjacencyMatrix in C++ backend #513
base: main
Are you sure you want to change the base?
Conversation
So, while adding support any data structure in C++ backend we need to make sure that there is no difference between the interface supported by C++ backend and Python backend. Internally both can follow different implementations. Since we have already implemented Graph in Python backend so we need to make sure that interface of every method defined in https://github.com/codezonediitj/pydatastructs/blob/master/pydatastructs/graphs/graph.py works with C++ backend as well. The pattern I notice is here that we pass nodes as input, call So I would say let's follow a similar pattern for now in C++ backend as well, with a small tweak i.e., I would use Now,
And in addition to
|
Codecov Report
@@ Coverage Diff @@
## master #513 +/- ##
=============================================
+ Coverage 98.528% 98.538% +0.009%
=============================================
Files 32 34 +2
Lines 4010 4037 +27
=============================================
+ Hits 3951 3978 +27
Misses 59 59
|
Add graph cpp backend.
Currently working on AdjacencyMatrix implementation.