documentation-website icon indicating copy to clipboard operation
documentation-website copied to clipboard

Adding median absolute deviation documentation page

Open MitchellGale opened this issue 10 months ago • 5 comments

Description

Adds documentation for the median_absolute_deviation aggregation function.

Issues Resolved

https://github.com/opensearch-project/documentation-website/issues/4293

Checklist

  • [x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin. For more information on following Developer Certificate of Origin and signing off your commits, please check here.

MitchellGale avatar Mar 28 '24 21:03 MitchellGale

@martin-gaievski - Can you sign off on this from a technical standpoint? Thanks.

hdhalter avatar Mar 29 '24 20:03 hdhalter

This may need more context regarding the compression field. https://github.com/opensearch-project/OpenSearch/blob/ef2a9e5a290262201f069539de45248e1bfe34e4/server/src/main/java/org/opensearch/search/aggregations/metrics/MedianAbsoluteDeviationAggregationBuilder.java#L106

GumpacG avatar Apr 02 '24 22:04 GumpacG

@sandeshkr419 - Can you review this for technical accuracy?

hdhalter avatar Apr 11 '24 22:04 hdhalter

@sandeshkr419 - Do you approve of this content from a technical standpoint?

@vagimeli for visibility, as you are working on aggregations content.

hdhalter avatar May 02 '24 00:05 hdhalter

@MitchellGale Thanks for drafting up the doc. I have some comments on adding more details to few sections for better understanding.

sandeshkr419 avatar May 02 '24 02:05 sandeshkr419

Doc review complete. I'll move this to editorial once suggestions are addressed.

@MitchellGale @sandeshkr419 Please let me know know once the doc review changes are accepted. This will need to go through editorial review before we can merge it. I'll handle that process. We just need an SME's tech review approval before I can move this forward. Thank you.

vagimeli avatar May 15 '24 14:05 vagimeli