apache / apache/arrow

[Python][C++] Update type definition of npy_traits for Float16 to arrow::util::Float16 instead of uint16_t

Open
#46,701 1 comment 0 reactions 0 assignees View on GitHub
Component: C++ Component: Python Status: needs champion 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 suggested on the following comment:
https://github.com/apache/arrow/pull/46618#discussion_r2112073163

The type definition of `npy_traits` for Float16 could be `arrow::util::Float16` instead of `uint16_t`, making it semantically clearer.

https://github.com/apache/arrow/blob/55628cd5fed83187d65b8860c3edfc70e09f9a57/python/pyarrow/src/arrow/python/type_traits.h#L88-L90

### Component(s)

C++, Python

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.