fiware-sth-comet icon indicating copy to clipboard operation
fiware-sth-comet copied to clipboard

Published 20 hours ago • verified publisher icon telefonicaid

Improved aggrMethod to accept multiple values

Open Madhu1029 opened this issue 2 years ago • 7 comments

Fixes #432 partially. aggrMethod can take multiple aggregations using comma separation.

Madhu1029 avatar Feb 22 '23 12:02 Madhu1029

Hi @fgalan ,

If PR seems ok, please merge the PR. Thanks

Madhu1029 avatar Mar 16 '23 06:03 Madhu1029

Hi @fgalan , I have updated the docs. Please merge this PR if it seems ok.

Thanks

Madhu1029 avatar May 12 '23 11:05 Madhu1029

Hi @fgalan ,

I have updated all the suggestions mentioned by you in previous comment. I request you to please review the PR.

Madhu1029 avatar Oct 03 '23 04:10 Madhu1029

Hi @fgalan ,

I have updated all the suggestions mentioned by you in previous comment. I request you to please review the PR.

Thanks for your contribution! We would have a look to the PR the soon as possible.

fgalan avatar Oct 03 '23 15:10 fgalan

Hi @fgalan ,

Could you please have a look on this PR?

Madhu1029 avatar Apr 12 '24 04:04 Madhu1029

Is this a new feature or extends and modify a current one? Should documentation be updated ?

AlvaroVega avatar Apr 12 '24 09:04 AlvaroVega

Is this a new feature or extends and modify a current one? Should documentation be updated ?

Hi @AlvaroVega , Thanks for your response. Currently, we can only specify single aggrMethod like aggrMethod=sum or aggrMethod=min but with this PR we can specify multiple aggrMethod in single go like aggrMethod=sum,min,max. In addition to this, aggrMethod=all will give all the aggregation method results in one go.

Documentation for this is already updated in this PR.

Madhu1029 avatar Apr 12 '24 09:04 Madhu1029