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

feat: direct interface for active.cc and variable rename for understandability #4671

Merged
merged 11 commits into from
Mar 7, 2024

Merge branch 'master' into master

5907c04
Select commit
Loading
Failed to load commit list.
Merged

feat: direct interface for active.cc and variable rename for understandability #4671

Merge branch 'master' into master
5907c04
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Feb 21, 2024 in 0s

43.58% of diff hit (target 81.50%)

View this Pull Request on Codecov

43.58% of diff hit (target 81.50%)

Annotations

Check warning on line 124 in vowpalwabbit/core/src/reductions/active.cc

See this annotation in the file changed.

@codecov codecov / codecov/patch

vowpalwabbit/core/src/reductions/active.cc#L124

Added line #L124 was not covered by tests

Check warning on line 127 in vowpalwabbit/core/src/reductions/active.cc

See this annotation in the file changed.

@codecov codecov / codecov/patch

vowpalwabbit/core/src/reductions/active.cc#L126-L127

Added lines #L126 - L127 were not covered by tests

Check warning on line 129 in vowpalwabbit/core/src/reductions/active.cc

See this annotation in the file changed.

@codecov codecov / codecov/patch

vowpalwabbit/core/src/reductions/active.cc#L129

Added line #L129 was not covered by tests

Check warning on line 132 in vowpalwabbit/core/src/reductions/active.cc

See this annotation in the file changed.

@codecov codecov / codecov/patch

vowpalwabbit/core/src/reductions/active.cc#L131-L132

Added lines #L131 - L132 were not covered by tests

Check warning on line 134 in vowpalwabbit/core/src/reductions/active.cc

See this annotation in the file changed.

@codecov codecov / codecov/patch

vowpalwabbit/core/src/reductions/active.cc#L134

Added line #L134 was not covered by tests

Check warning on line 143 in vowpalwabbit/core/src/reductions/active.cc

See this annotation in the file changed.

@codecov codecov / codecov/patch

vowpalwabbit/core/src/reductions/active.cc#L138-L143

Added lines #L138 - L143 were not covered by tests

Check warning on line 150 in vowpalwabbit/core/src/reductions/active.cc

See this annotation in the file changed.

@codecov codecov / codecov/patch

vowpalwabbit/core/src/reductions/active.cc#L149-L150

Added lines #L149 - L150 were not covered by tests

Check warning on line 152 in vowpalwabbit/core/src/reductions/active.cc

See this annotation in the file changed.

@codecov codecov / codecov/patch

vowpalwabbit/core/src/reductions/active.cc#L152

Added line #L152 was not covered by tests

Check warning on line 276 in vowpalwabbit/core/src/reductions/active.cc

See this annotation in the file changed.

@codecov codecov / codecov/patch

vowpalwabbit/core/src/reductions/active.cc#L271-L276

Added lines #L271 - L276 were not covered by tests