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

Implement Bixby and Wagner's Almost Linear-Time Algorithm for Graph Realization #20834

Open
TaraFife mannequin opened this issue Jun 16, 2016 · 25 comments
Open

Implement Bixby and Wagner's Almost Linear-Time Algorithm for Graph Realization #20834

TaraFife mannequin opened this issue Jun 16, 2016 · 25 comments

Comments

@TaraFife
Copy link
Mannequin

TaraFife mannequin commented Jun 16, 2016

Currently, the plan is to create a new class for the decomposition,
and use this to rewrite the is_graphic function, so that we can
return a graph that realizes the matroid.

CC: @sagetrac-Stefan @sagetrac-yomcat

Component: matroid theory

Branch/Commit: u/gh-neilthiessen/graphicness_test @ 6332117

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

@TaraFife TaraFife mannequin added this to the sage-7.3 milestone Jun 16, 2016
@TaraFife
Copy link
Mannequin Author

TaraFife mannequin commented Jun 16, 2016

Branch: u/tara/graphicness_test

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 20, 2016

Commit: 1d85d1c

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 20, 2016

Branch pushed to git repo; I updated commit sha1. New commits:

1d85d1cImplamented additional functions.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 29, 2016

Branch pushed to git repo; I updated commit sha1. New commits:

2d7255dAdded functions, found bugs, still incomplete.
d2192caFile is closer to complete now.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 29, 2016

Changed commit from 1d85d1c to d2192ca

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jul 11, 2016

Changed commit from d2192ca to edb861d

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jul 11, 2016

Branch pushed to git repo; I updated commit sha1. New commits:

edb861dAdded documentation fixed errors

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jul 14, 2016

Changed commit from edb861d to 5ac8637

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jul 14, 2016

Branch pushed to git repo; I updated commit sha1. New commits:

5ac8637Updated merge functions

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jul 19, 2016

Branch pushed to git repo; I updated commit sha1. New commits:

89e59efImplamented ``squeeze`` function.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jul 19, 2016

Changed commit from 5ac8637 to 89e59ef

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 10, 2016

Branch pushed to git repo; I updated commit sha1. New commits:

279aba1added get D_hat

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 10, 2016

Changed commit from 89e59ef to 279aba1

@sagetrac-Stefan
Copy link
Mannequin

sagetrac-Stefan mannequin commented Aug 12, 2016

comment:8

Just a quick observation as I'm looking at the current code: the filename should be changed to reflect the actual authors of the algorithm.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 12, 2016

Changed commit from 279aba1 to ba510d2

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 12, 2016

Branch pushed to git repo; I updated commit sha1. New commits:

ba510d2Infinate recursion?

@TaraFife
Copy link
Mannequin Author

TaraFife mannequin commented Aug 12, 2016

comment:10

Replying to @sagetrac-Stefan:

Just a quick observation as I'm looking at the current code: the filename should be changed to reflect the actual authors of the algorithm.

Yep.

@TaraFife
Copy link
Mannequin Author

TaraFife mannequin commented Aug 12, 2016

comment:11

Replying to @sagetrac-git:

Branch pushed to git repo; I updated commit sha1. New commits:

ba510d2Infinate recursion?

I don't know how I introduced this runtime error. The function does need to be reworked to make sure that any of the edges in Z, which correspond to parent markers of type 2, 3, or 4 children are on the end(s) of the path.

@neilthiessen neilthiessen self-assigned this Jan 24, 2020
@neilthiessen
Copy link

Changed branch from u/tara/graphicness_test to u/gh-neilthiessen/graphicness_test

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 6, 2020

Changed commit from ba510d2 to 9754232

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 6, 2020

Branch pushed to git repo; I updated commit sha1. New commits:

9754232Small changes to make the build complete with the current version of sage.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Sep 20, 2021

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

4ed85c2This is very incomplete code. It has a lot of stubs.
7b0cd24Implamented additional functions.
e8a7089Added functions, found bugs, still incomplete.
12fd703File is closer to complete now.
77f98acAdded documentation fixed errors
e30edfbUpdated merge functions
6b0d17eImplamented ``squeeze`` function.
0021426added get D_hat
c3b6698Infinate recursion?
6332117Small changes to make the build complete with the current version of sage.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Sep 20, 2021

Changed commit from 9754232 to 6332117

@slel
Copy link
Member

slel commented Sep 21, 2021

comment:16

Remember to fill in your full name in the
"Authors" field, and to set to "needs review".

@slel

This comment has been minimized.

@mkoeppe mkoeppe removed this from the sage-7.3 milestone Dec 29, 2022
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

3 participants