Skip to content
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

Multiflow #8870

Closed
nathanncohen mannequin opened this issue May 4, 2010 · 12 comments
Closed

Multiflow #8870

nathanncohen mannequin opened this issue May 4, 2010 · 12 comments

Comments

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented May 4, 2010

Multiflow in Sage :-)

http://en.wikipedia.org/wiki/Multi-commodity_flow_problem

Nathann

Component: graph theory

Author: Nathann Cohen

Reviewer: Robert Miller

Merged: sage-4.5.alpha1

Issue created by migration from https://trac.sagemath.org/ticket/8870

@nathanncohen nathanncohen mannequin added this to the sage-4.5 milestone May 4, 2010
@nathanncohen

This comment has been minimized.

@nathanncohen nathanncohen mannequin added the s: needs review label May 8, 2010
@nathanncohen

This comment has been minimized.

@nathanncohen

This comment has been minimized.

@nathanncohen nathanncohen mannequin unassigned jasongrout and rlmill May 8, 2010
@nathanncohen
Copy link
Mannequin Author

nathanncohen mannequin commented May 8, 2010

comment:4

In this patch, the method multi-commodity flow is defined. The code is also refactored as this method shares many common points with flow. The new code is more compact and (I hope) easier to understand ! :-)

Nathann

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jun 21, 2010

comment:5
  1. Needs rebasing (applying on top of clean up #optional tags in sage/graphs #9269, Overfull graph (and a bug in edge_coloring) #8781, and Broken docstrings in Travelling Salesman Problem #9230).

  2. # optional tags need to follow the correct format.

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jun 21, 2010

Author: Nathann Cohen

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jun 21, 2010

Reviewer: Robert Miller

@rlmill rlmill mannequin added s: needs work and removed s: needs review labels Jun 21, 2010
@nathanncohen
Copy link
Mannequin Author

nathanncohen mannequin commented Jun 21, 2010

comment:6

Updated ! A nasty piece to rebase... ;-)

Nathann

@nathanncohen

This comment has been minimized.

@nathanncohen
Copy link
Mannequin Author

nathanncohen mannequin commented Jun 21, 2010

Attachment: trac_8870.patch.gz

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jun 21, 2010

comment:7

Attachment: trac_8870-ref.patch.gz

Looks good to me. Apply both patches.

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jun 29, 2010

Merged: sage-4.5.alpha1

@rlmill rlmill mannequin removed the s: positive review label Jun 29, 2010
@rlmill rlmill mannequin closed this as completed Jun 29, 2010
@nathanncohen nathanncohen mannequin mentioned this issue Jun 29, 2010
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant