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.

Sign up for GitHub

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

Jump to bottom

Move sage/graphs/planarity/ to sage/graphs/planarity_c/ #13363

Closed
mwhansen opened this issue Aug 13, 2012 · 4 comments
Closed

Move sage/graphs/planarity/ to sage/graphs/planarity_c/ #13363

mwhansen opened this issue Aug 13, 2012 · 4 comments
Assignees
@jasongrout @rlmill @nathanncohen
Labels
c: graph theory p: major / 3 t: bug
Milestone
sage-5.3

Comments

@mwhansen
Copy link
Contributor

When running doctests with #12659 applied, an "ImportWarning: Not importing directory" is raised due to there being a sage/graphs/planarity.so and sage/graphs/planarity directory. This moves the C source code into a planarity_c directory.

Component: graph theory

Author: Mike Hansen

Reviewer: John Palmieri

Merged: sage-5.3.beta2

Issue created by migration from https://trac.sagemath.org/ticket/13363
@mwhansen mwhansen added this to the sage-5.3 milestone Aug 13, 2012
@mwhansen
Copy link
Contributor Author

Attachment: trac_13363.patch.gz

@jhpalmieri
Copy link
Member

Reviewer: John Palmieri

@jhpalmieri
Copy link
Member

comment:2

Looks good to me.

@jdemeyer
Copy link

Merged: sage-5.3.beta2

@mwhansen mwhansen mentioned this issue Sep 13, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jasongrout jasongrout

@rlmill rlmill

@nathanncohen nathanncohen

Labels
c: graph theory p: major / 3 t: bug
Projects
None yet
Milestone
sage-5.3
Development

No branches or pull requests
4 participants
@jasongrout @mwhansen @jhpalmieri @jdemeyer