Skip to content

Conversation

@dongxiao1198
Copy link
Contributor

feat: add inclusive metrics evaluator

@dongxiao1198 dongxiao1198 force-pushed the inclusive_evaluator branch 2 times, most recently from 529234f to b1ebcab Compare November 28, 2025 08:15
Copy link
Contributor

@HeartLinked HeartLinked left a comment

Choose a reason for hiding this comment

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

Modify meson files too?

}
};

auto data_file = std::make_shared<DataFile>();
Copy link
Contributor

Choose a reason for hiding this comment

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

You can use designated initializers here.

@dongxiao1198
Copy link
Contributor Author

Modify meson files too?

done

Copy link
Member

@wgtmac wgtmac left a comment

Choose a reason for hiding this comment

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

+1

@wgtmac wgtmac merged commit 212f087 into apache:main Dec 2, 2025
10 checks passed
@dongxiao1198 dongxiao1198 deleted the inclusive_evaluator branch December 3, 2025 01:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants