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

Feature/reduce-decoder-mem-usage #84

Draft
wants to merge 3 commits into
base: develop
Choose a base branch
from
Draft

Feature/reduce-decoder-mem-usage #84

wants to merge 3 commits into from

Commits on Nov 21, 2024

  1. reduce decoder mem usage 

    declare an empty accum tensor outside the for loop. the old way of having out and out1 results in two copies of the array which results in more memory use. at 9km this added 6gb to peak mem usage
    @cathalobrien
    cathalobrien committed Nov 21, 2024
    Configuration menu
    Copy the full SHA
    a12b4f2 View commit details
    Browse the repository at this point in the history

  2. typo

    @cathalobrien
    cathalobrien committed Nov 21, 2024
    Configuration menu
    Copy the full SHA
    f7ce093 View commit details
    Browse the repository at this point in the history

  3. [pre-commit.ci] auto fixes from pre-commit.com hooks 

    for more information, see https://pre-commit.ci
    @pre-commit-ci
    pre-commit-ci[bot] committed Nov 21, 2024
    Configuration menu
    Copy the full SHA
    a81ab4e View commit details
    Browse the repository at this point in the history