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

Use liftstd for BasisOfRowsCoeff #476

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Apr 27, 2022

  1. Use liftstd for BasisOfRowsCoeff

    The runtime of the added example drops from 25s to 6s on my computer.
    
    Notes:
    * This commit is very unfinished. Its only purpose is to show the potential
      and for quick tests.
    * `liftstd(..., "std")` has the same output as `std` but only up to
      clearing denominators, so one would also have to adjust `std`, `syz` etc.
    * Using the relative version of `liftstd` one could also implement a more
      efficient `RelativeBasisOfRowsCoeff`.
    * https://github.com/zickgraf/homalg_project/tree/use_liftstd has some more
      experiments in this direction.
    zickgraf committed Apr 27, 2022
    Configuration menu
    Copy the full SHA
    519538a View commit details
    Browse the repository at this point in the history