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.

Sign up for GitHub

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

Jump to bottom

Add alias for InTimeAccumulateWeightedAggregator #1117

Open
holgerroth opened this issue Nov 8, 2022 · 2 comments
Open

Add alias for InTimeAccumulateWeightedAggregator #1117

holgerroth opened this issue Nov 8, 2022 · 2 comments
Assignees
@chesterxgchen @IsaacYangSLA @YuanTingHsieh
Labels
enhancement New feature or request

Comments

@holgerroth
Copy link
Collaborator

Feature request
InTimeAccumulateWeightedAggregator has become our go-to aggregator. However, the name is quite long. A shorter alias could be useful, like WeightedAggregator.
@holgerroth holgerroth added the bug Something isn't working label Nov 8, 2022
@holgerroth holgerroth added enhancement New feature or request and removed bug Something isn't working labels Nov 8, 2022
@chesterxgchen
Copy link
Collaborator

Yes we should should this. I think @holgerroth or @YuanTingHsieh could create a PR to rename. But probably have to change all the codes including examples

@holgerroth
Copy link
Collaborator Author

Just adding an alias so new examples can use the short name would be a start.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chesterxgchen chesterxgchen

@IsaacYangSLA IsaacYangSLA

@YuanTingHsieh YuanTingHsieh

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@chesterxgchen @IsaacYangSLA @holgerroth @YuanTingHsieh