Skip to content

Conversation

jan-elastic
Copy link
Contributor

@jan-elastic jan-elastic commented Sep 5, 2024

For hyperbolic functions "an angle, in radians" makes no sense (radians are a unit of circular angle, which is not applicable for hyperbolic geometry).

See also Wikipedia or the underlying function's JavaDocs.

@elasticsearchmachine elasticsearchmachine added v8.16.0 needs:triage Requires assignment of a team area label labels Sep 5, 2024
@jan-elastic jan-elastic added >non-issue :Analytics/ES|QL AKA ESQL and removed needs:triage Requires assignment of a team area label labels Sep 5, 2024
@jan-elastic jan-elastic requested a review from nik9000 September 5, 2024 15:27
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Sep 5, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@alex-spies alex-spies self-requested a review September 6, 2024 07:25
Copy link
Contributor

@alex-spies alex-spies left a comment

Choose a reason for hiding this comment

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

Nice. Thank you, fellow maths afficionado!

@astefan
Copy link
Contributor

astefan commented Sep 6, 2024

Slightly related: #112437
Thank you @jan-elastic for pro-actively looking at these and updating them 🙏 .

Copy link
Contributor

@astefan astefan left a comment

Choose a reason for hiding this comment

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

LGTM

@jan-elastic jan-elastic force-pushed the fix-esql-hyperbolic-fns-docs branch from 85f429d to e6f4b3f Compare September 6, 2024 09:21
@jan-elastic jan-elastic merged commit f8a7822 into main Sep 6, 2024
1 check passed
@jan-elastic jan-elastic deleted the fix-esql-hyperbolic-fns-docs branch September 6, 2024 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL >non-issue Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants