-
Notifications
You must be signed in to change notification settings - Fork 1
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 Directionality into MCF #8
base: main
Are you sure you want to change the base?
Conversation
for graph 3 and 7, I was expected the same edges to be on the output, but that did not happen. *reminder to see why later |
with the updates to the code, graph 3 and 7 are still not the same. I think at this point, I think the solver may be doing something behind the scenes. |
Let's make a v0.1 release before merging this so that this version of MCF can become v0.2. That will make it easier for the Dockerfile in SPRAS to refer to versioned releases of this code. |
Ready for code review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I finally starting reviewing these changes. I only looked at the code changes so far. I would like to run the updated version locally and inspect all the test graphs before merging.
Ready for code review again |
No description provided.