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

e52ca46
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
e52ca46
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Mar 5, 2024 in 0s

47.22% of diff hit (target 81.64%)

View this Pull Request on Codecov

47.22% of diff hit (target 81.64%)

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 131 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

Added line #L131 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L133 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

vowpalwabbit/core/src/reductions/active.cc#L137-L140

Added lines #L137 - L140 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

vowpalwabbit/core/src/reductions/active.cc#L146-L147

Added lines #L146 - L147 were not covered by tests

Check warning on line 149 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

Added line #L149 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

vowpalwabbit/core/src/reductions/active.cc#L268-L273

Added lines #L268 - L273 were not covered by tests