-
Notifications
You must be signed in to change notification settings - Fork 4
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
Graphs interface #38
Comments
Cool! @olegfafurin please feel free to make issues here for literally any problems you have or missing features you find. That kind of feedback is really useful at this stage. Note that #37 will change testing and implementation definition a little and #34 will allow (multiple) inheritance soon. |
The interface is more or less complete for the time being, and the tests have grown larger, in case you're interested to see your package in action @rafaqz |
The tests look good! Happy its useful :) |
Also nice use of readme as the interface description! Do you imagine packages will have this as a dependency, or just for checking the interface? |
For now it's a prototype, still waiting to evolve. Ideally I'd put this in Graphs.jl, but I fear there might be some pushback. |
Yes abosolutely. Packages could just have it as a test dependency. |
My intern @olegfafurin and I are trying to formalize the
AbstractGraph
interface in this repo: https://github.com/gdalle/GraphsInterfaceChecker.jlWe might poke around and find missing stuff in Interfaces.jl
The text was updated successfully, but these errors were encountered: