Skip to content

Conversation

@nik9000
Copy link
Member

@nik9000 nik9000 commented Oct 30, 2018

The random_score function produces values between 0 (inclusive) and 1
(exclusive) and documented it with fancy methematical range notation. It
is so fancy I thought it was a typo. This changes the documentation to
use words.

Relates to #35084

The `random_score` function produces values between 0 (inclusive) and 1
(exclusive) and documented it with fancy methematical range notation. It
is so fancy I thought it was a typo. This changes the documentation to
use words.

Relates to elastic#35084
@nik9000 nik9000 added >docs General docs changes :Search/Search Search-related issues that do not fall into other categories v7.0.0 v6.6.0 v6.4.4 v6.5.1 labels Oct 30, 2018
@nik9000 nik9000 requested a review from colings86 October 30, 2018 16:21
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search-aggs

Copy link
Contributor

@colings86 colings86 left a comment

Choose a reason for hiding this comment

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

LGTM

@nik9000 nik9000 merged commit 2b2e208 into elastic:master Oct 30, 2018
nik9000 added a commit that referenced this pull request Oct 30, 2018
The `random_score` function produces values between 0 (inclusive) and 1
(exclusive) and documented it with fancy methematical range notation. It
is so fancy I thought it was a typo. This changes the documentation to
use words.

Relates to #35084
nik9000 added a commit that referenced this pull request Oct 30, 2018
The `random_score` function produces values between 0 (inclusive) and 1
(exclusive) and documented it with fancy methematical range notation. It
is so fancy I thought it was a typo. This changes the documentation to
use words.

Relates to #35084
nik9000 added a commit that referenced this pull request Oct 30, 2018
The `random_score` function produces values between 0 (inclusive) and 1
(exclusive) and documented it with fancy methematical range notation. It
is so fancy I thought it was a typo. This changes the documentation to
use words.

Relates to #35084
@nik9000
Copy link
Member Author

nik9000 commented Oct 30, 2018

Thanks for reviewing @colings86!

@mayya-sharipova
Copy link
Contributor

I also always thought this was a typo :)

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

Labels

>docs General docs changes :Search/Search Search-related issues that do not fall into other categories v6.4.3 v6.5.0 v6.6.0 v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants