apache / apache/arrow

[C++] Half-float scalars should use Float16 for value type, not uint16_t

Open
#46,858 3 comments 0 reactions 1 assignee Claimed by @benibus View on GitHub
Component: C++ Type: enhancement
Dominant language
C++
Stars
17.1k
Forks
4.3k
Avg merge
3d 13h
Merged PRs (30d)
88

Description

### Describe the enhancement requested

As the title says. Currently, `HalfFloatScalar::value` is a `uint16_t`, but it should be `util::Float16` instead.

This is a slightly compatibility-breaking change, but it's for the better.

### Component(s)

C++

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.