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

Add keyword argument for contract operators, clean simulation pipeline #287

Merged

lukem12345 merged 5 commits into main from llm/no-mat-mul

Dec 9, 2024

Merged

Add keyword argument for contract operators, clean simulation pipeline #287

Encapsulate, remove dead code, simplify control flow

Codecov / codecov/patch succeeded Dec 7, 2024 in 0s

91.37% of diff hit (target 69.00%)

View this Pull Request on Codecov

91.37% of diff hit (target 69.00%)

Annotations

Check warning on line 210 in src/simulation.jl

codecov / codecov/patch

src/simulation.jl#L210

Added line #L210 was not covered by tests

Check warning on line 568 in src/simulation.jl

codecov / codecov/patch

src/simulation.jl#L568

Added line #L568 was not covered by tests

Check warning on line 573 in src/simulation.jl

codecov / codecov/patch

src/simulation.jl#L573

Added line #L573 was not covered by tests

Check warning on line 667 in src/simulation.jl

codecov / codecov/patch

src/simulation.jl#L666-L667

Added lines #L666 - L667 were not covered by tests

View more details on Codecov