Skip to content

gh-146238: support half-floats in the array module - #146242

Merged
vstinner merged 6 commits into
python:mainfrom
skirpichev:support-halffloat-arrays/146238
Mar 30, 2026
Merged

gh-146238: support half-floats in the array module#146242
vstinner merged 6 commits into
python:mainfrom
skirpichev:support-halffloat-arrays/146238

Conversation

@skirpichev

@skirpichevskirpichev commented Mar 21, 2026•
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Member

@bedevere-appbedevere-appBot mentioned this pull request Mar 21, 2026
Closed
@skirpichev

This comment was marked as resolved.

Sign in to view
@skirpichev
skirpichevforce-pushed the support-halffloat-arrays/146238 branch from ac5646a to 2bac9b5CompareMarch 26, 2026 19:46
@skirpichev
skirpichev marked this pull request as ready for review March 26, 2026 23:05
@hugovk
hugovk removed their request for review March 27, 2026 11:34
vstinner
vstinner reviewed Mar 27, 2026
View reviewed changes
Comment threadDoc/library/array.rst Outdated
Comment threadDoc/whatsnew/3.15.rst Outdated
skirpichev
skirpichev commented Mar 28, 2026
View reviewed changes
Comment threadDoc/whatsnew/3.15.rst Outdated
skirpichevand others added 2 commits March 28, 2026 03:28
@skirpichev@vstinner
Apply suggestions from code review
b98ebb0
Co-authored-by: Victor Stinner <vstinner@python.org>
Co-authored-by: Sergey B Kirpichev <skirpichev@gmail.com>
@skirpichev
Merge branch 'main' into support-halffloat-arrays/146238
0702cb3
vstinner
vstinner approved these changes Mar 30, 2026
View reviewed changes

@vstinnervstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. I have a minor suggestion on the test.

Comment threadLib/test/test_array.py Outdated
@vstinner
vstinner enabled auto-merge (squash) March 30, 2026 11:27
@vstinner

vstinner commented Mar 30, 2026

Copy link
Copy Markdown
Member

Hum. It seems like the CI is stuck :-(

 All required checks passExpected — Waiting for status to be reported
DO-NOT-MERGEExpected — Waiting for status to be reported
Unresolved reviewExpected — Waiting for status to be reported

I clicked on [Update branch] to try to unblock the CI.

@vstinner

vstinner commented Mar 30, 2026

Copy link
Copy Markdown
Member

There are test failures. Example on Windows x64: https://github.com/python/cpython/actions/runs/23743340664/job/69165604606?pr=146242


FAIL: test_cmp (test.test_array.HalfFloatTest.test_cmp)
----------------------------------------------------------------------
Traceback (most recent call last):
File "D:\a\cpython\cpython\Lib\test\test_array.py", line 624, in test_cmp
self.assertIs(a < ab, True)
~~~~~~~~~~~~~^^^^^^^^^^^^^^
AssertionError: False is not True

@skirpichev

skirpichev commented Mar 30, 2026

Copy link
Copy Markdown
MemberAuthor

There are test failures.

e8bb6f3 is the problem: biggerexample[-2] now is smaller than example[-2] for 'e' type.

vstinner
vstinner reviewed Mar 30, 2026
View reviewed changes
Comment threadLib/test/test_array.py Outdated
@vstinner

vstinner commented Mar 30, 2026

Copy link
Copy Markdown
Member

e8bb6f3 is the problem: biggerexample[-2] now is smaller than example[-2] for 'e' type.

Ooops. I fixed test_array in the following commit.

@vstinner

vstinner commented Mar 30, 2026

Copy link
Copy Markdown
Member

I modified example, smallerexample and biggerexample values of HalfFloatTest to use values closer to FPTest.

@vstinner
vstinner merged commit e79fd60 into python:mainMar 30, 2026
53 checks passed
@vstinner

vstinner commented Mar 30, 2026

Copy link
Copy Markdown
Member

Merged, thanks for your contribution.

@skirpichev
skirpichev deleted the support-halffloat-arrays/146238 branch March 30, 2026 22:41
maurycy added a commit to maurycy/cpython that referenced this pull request Mar 31, 2026
@maurycy
Merge branch 'main' into tachyon-ndjson-kolektor
9836ffa
* main:
pythongh-145458: use `self.skip_idle` consistently in the tachyon profiler (python#145459)
pythongh-146615: Fix format specifiers in Objects/ directory (pythonGH-146620)
pythongh-146615: Fix format specifiers in Python/ directory (pythonGH-146619)
pythongh-146615: Fix format specifiers in test cextensions (pythonGH-146618)
pythongh-146615: Fix format specifiers in extension modules (pythonGH-146617)
pythongh-146615: Fix crash in __get__() for METH_METHOD descriptors with invalid type argument (pythonGH-146634)
pythongh-146376: Reduce timeout in Emscripten GHA workflow (python#146378)
pythongh-146442: Fix various bugs in compiler pipeline (python#146443)
pythongh-146238: Support half-floats in the array module (python#146242)
pythongh-145056: Add support for merging collections.UserDict and frozendict (pythonGH-146465)
pythongh-145056: Fix merging of collections.OrderedDict and frozendict (pythonGH-146466)
pythongh-139633: Run netrc file permission check only once per parse (pythonGH-139634)
@pablogsalpablogsal mentioned this pull request Apr 4, 2026
Merged
clin1234 pushed a commit to clin1234/cpython that referenced this pull request Apr 16, 2026
@skirpichev@vstinner@clin1234
pythongh-146238: Support half-floats in the array module (python#146242)
8286d51
Co-authored-by: Victor Stinner <vstinner@python.org>
ljfp pushed a commit to ljfp/cpython that referenced this pull request Apr 25, 2026
@skirpichev@vstinner@ljfp
pythongh-146238: Support half-floats in the array module (python#146242)
906aff2
Co-authored-by: Victor Stinner <vstinner@python.org>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vstinnervstinnervstinner approved these changes
@AA-TurnerAA-TurnerAwaiting requested review from AA-TurnerAA-Turner is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@skirpichev@vstinner