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

slightly faster Moebius matrix for posets #28551

Closed
fchapoton opened this issue Oct 3, 2019 · 7 comments
Closed

slightly faster Moebius matrix for posets #28551

fchapoton opened this issue Oct 3, 2019 · 7 comments

Comments

@fchapoton
Copy link
Contributor

CC: @tscrim @jm58660

Component: combinatorics

Author: Frédéric Chapoton

Branch/Commit: dece8f6

Reviewer: Travis Scrimshaw

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

@fchapoton fchapoton added this to the sage-9.0 milestone Oct 3, 2019
@fchapoton
Copy link
Contributor Author

Commit: dece8f6

@fchapoton
Copy link
Contributor Author

New commits:

dece8f6faster Moebius matrix for posets

@fchapoton
Copy link
Contributor Author

Branch: u/chapoton/28551

@fchapoton
Copy link
Contributor Author

comment:2

Before:

sage: P=posets.TamariLattice(7)
sage: %time P.moebius_function_matrix()
CPU times: user 606 ms, sys: 553 µs, total: 607 ms
Wall time: 606 ms
429 x 429 dense matrix over Integer Ring (use the '.str()' method to see the entries)

After:

sage: P=posets.TamariLattice(7)
sage: %time P.moebius_function_matrix()
CPU times: user 359 ms, sys: 3.85 ms, total: 362 ms
Wall time: 362 ms
429 x 429 dense matrix over Integer Ring (use the '.str()' method to see the entries)

@tscrim
Copy link
Collaborator

tscrim commented Oct 3, 2019

comment:3

It is a more memory intensive version, but I don't think it is too significant, especially compared to the speedup. LGTM.

@tscrim
Copy link
Collaborator

tscrim commented Oct 3, 2019

Reviewer: Travis Scrimshaw

@vbraun
Copy link
Member

vbraun commented Oct 6, 2019

Changed branch from u/chapoton/28551 to dece8f6

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