-
Notifications
You must be signed in to change notification settings - Fork 8
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
Node ordering #19
Node ordering #19
Conversation
Codecov Report
@@ Coverage Diff @@
## main #19 +/- ##
==========================================
+ Coverage 92.09% 92.22% +0.13%
==========================================
Files 4 4
Lines 177 180 +3
==========================================
+ Hits 163 166 +3
Misses 14 14
Continue to review full report at Codecov.
|
this will now need rebasing as i merged your other PR |
@oxinabox Done; I also set the input type to be Vector{Pair} |
Nice LGTM |
This commit attempts to solve the issue #12 to set the ordering of selected nodes.
This branch is also the updated version of previous #17 ; it is built on top of the pull request #16 where the tests are modified to accommodate optional arguments.