Skip to content

C++: Update stats file after DCA and extractor changes#19870

Merged
jketema merged 1 commit intogithub:mainfrom
jketema:jketema/stats
Jun 26, 2025
Merged

C++: Update stats file after DCA and extractor changes#19870
jketema merged 1 commit intogithub:mainfrom
jketema:jketema/stats

Conversation

@jketema
Copy link
Copy Markdown
Contributor

@jketema jketema commented Jun 25, 2025

It's not completely clear to me why there are differences in extraction errors for abseil in DCA, as this PR clearly did not change anything in the extractor. I'll keep an eye on it while reviewing the nighties; I don't think this should block merging this.

For the reviewer: you want to look at the top most DCA experiment that is linked here.

@github-actions github-actions Bot added the C++ label Jun 25, 2025
@jketema jketema marked this pull request as ready for review June 25, 2025 13:52
Copilot AI review requested due to automatic review settings June 25, 2025 13:52
@jketema jketema requested a review from a team as a code owner June 25, 2025 13:52
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

Copy link
Copy Markdown
Contributor

@IdrissRio IdrissRio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jketema jketema merged commit b16e710 into github:main Jun 26, 2025
11 checks passed
@jketema jketema deleted the jketema/stats branch June 26, 2025 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants