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

Add WeightedAvg aggregation #1771

Merged
merged 2 commits into from
Apr 22, 2020

Conversation

romainneutron
Copy link
Contributor

@romainneutron romainneutron commented Apr 21, 2020

Replace #1770

@ruflin
Copy link
Owner

ruflin commented Apr 22, 2020

@romainneutron Can you check CI?

@romainneutron
Copy link
Contributor Author

Hello,
It should be okay now.

@ruflin ruflin merged commit ea3502b into ruflin:master Apr 22, 2020
@ruflin
Copy link
Owner

ruflin commented Apr 22, 2020

@romainneutron Thank you for the contribution! Feel free to open a backport PR to 6.x if you need it here too.

@romainneutron romainneutron deleted the weighted-avg-agg-master branch April 22, 2020 11:28
@romainneutron
Copy link
Contributor Author

@ruflin Thanks! Done in #1773

ruflin pushed a commit that referenced this pull request Apr 22, 2020
ruflin pushed a commit that referenced this pull request Oct 20, 2020
This follows PR #1771 which added this aggregation but didn't expose it to DSL.
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.

None yet

2 participants