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

Less alloc in TriggsSdika filtering (fix #243) #244

Merged
merged 2 commits into from
Feb 9, 2022
Merged

Less alloc in TriggsSdika filtering (fix #243) #244

merged 2 commits into from
Feb 9, 2022

Commits on Feb 8, 2022

  1. Less alloc in TriggsSdika filtering (fix #243) 

    Putting the string interpolation behind a `@noinline` is the key change,
but it doesn't hurt to hoise the range creation.

Of the few remaining allocations, about half will be fixed by
JuliaLang/julia#43990, as they are another
instance of JuliaLang/julia#35972
    @timholy
    timholy committed Feb 8, 2022
    Configuration menu
    Copy the full SHA
    8b4a2f2 View commit details
    Browse the repository at this point in the history

  2. Don't use begin in indexing (Julia 1.0)

    @timholy
    timholy committed Feb 8, 2022
    Configuration menu
    Copy the full SHA
    41faa86 View commit details
    Browse the repository at this point in the history