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

Posets: Add is_series_parallel() #19215

Closed
jm58660 mannequin opened this issue Sep 15, 2015 · 12 comments
Closed

Posets: Add is_series_parallel() #19215

jm58660 mannequin opened this issue Sep 15, 2015 · 12 comments

Comments

@jm58660
Copy link
Mannequin

jm58660 mannequin commented Sep 15, 2015

Add a function to see if a poset is series-parallel (or "N-free").

First wait for #19659 to get closed. Series-parallel decomposition is just recursive applying of connected_components() and ordinal_sum_decomposition().

CC: @fchapoton

Component: combinatorics

Keywords: poset

Author: Jori Mäntysalo

Branch/Commit: b4b3254

Reviewer: Frédéric Chapoton

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

@jm58660 jm58660 mannequin added c: combinatorics labels Sep 15, 2015
@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Sep 15, 2015

comment:1

Do we agree that being series-parallel is not at all equivalent to being prime in the sense of modular decomposition?

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Sep 15, 2015

comment:2

Replying to @nathanncohen:

Do we agree that being series-parallel is not at all equivalent to being prime in the sense of modular decomposition?

Arghs, is_prime() was not what I think. So maybe there is no direct function to wrap. But anyway, if modular_decomposition() ends "withot any Prime", then the corresponding poset is N-free.

Thanks for correcting.

@jm58660

This comment has been minimized.

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Feb 10, 2016

comment:4

Now we have both pieces for decomposition. Next, what should be the return type be? modular_decomposition() of a graph returns a tuple of tuples of tuples etc. But is there some tree class ready for this?

"Horizontal decomposition", i.e. connected_components() is commutative, but "vertical decomposition" of course is not. But I guess we don't have a tree for this specific situation. Maybe Frédéric can comment on this.

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Apr 11, 2016

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Apr 11, 2016

Commit: b4b3254

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Apr 11, 2016

comment:6

I made at least the boolean (and maybe slow) version of this function. Anyways, better than nothing; a reviewer can check the documentation etc.


New commits:

b4b3254Added is_series_parallel().

@jm58660 jm58660 mannequin added the s: needs review label Apr 11, 2016
@saraedum
Copy link
Member

Author: Jori Mäntysalo

@fchapoton
Copy link
Contributor

Changed keywords from none to poset

@fchapoton fchapoton added this to the sage-7.2 milestone Apr 15, 2016
@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@fchapoton
Copy link
Contributor

comment:9

looks good to me

@vbraun
Copy link
Member

vbraun commented Apr 16, 2016

Changed branch from u/jmantysalo/posets__add_is_series_parallel__ to b4b3254

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