gh-150534: Add C23 half-turn trigonometric `*pi` functions by jepler · Pull Request #150555 · python/cpython · GitHub

See #150534 and https://discuss.python.org/t/add-half-turns-functions-cospi-et-al-to-math-module-following-c23-ieee754-2019/107550/5 for discussion and context.

The following functions are added to the math module:

acospi
asinpi
atanpi
atan2pi
cospi
sinpi
tanpi

The following aspects of a new feature are covered in this pull request:

Add tests
Add documentation including NEWS.d & whatsnew
Add configure-time checks for function presence
Regenerate configure

The math_testcases.txt cases are based on glibc's test cases, and their input & output values have been used. I supplemented this with tests of inf/nan arguments.

I extended math_testcases.txt so that it can test n-arg functions, and used it to test atan2pi.

Issue: Add 'half turns' functions (cospi et al) to math module following C23 & IEEE754-2019 #150534

Closes: #150534

* Add tests * Add documentation * Add configure-time checks for function presence * Regenerate configure No news item added, pending creation of a github issue. As committed, the local implementation of the functions is always used. For production, the #undefs should be removed so that the presence of the functions is detected by configure and used by mathmodule.

Documentation build overview

📚 cpython-previews | 🛠️ Build #33018618 | 📁 Comparing 4364ecb against main (f386f1f)

🔍 Preview build

98 files changed · + 1 added · ± 97 modified

+ Added

howto/abi3t-migration.html

± Modified

Please also test infinite/nan input and values with magnitude > 1. Include appropriate FPE flags, where it's required by the C23 Annex F.

With this, I think you can drop separate tests in the test_math.py for new functions.

Please add also a What's New entry.

Since the result is a float, should not we use 0.5 instead 0f 1/2? The same question for other documentation and examples.

That's a great question. These doc blocks were made by copypaste of the existing block (e.g., atan) then removing pi.

It's clear why an expression using pi is used in the existing atan2 docs. However, it's possible to ask which would be clearer in the atan2 comment: -.75*pi or -3*pi/4. I feel like the answer to this might make it clear whether -.75 or -3/4 is clearer in the new function's documentation.

Thoughts?

-3*pi/4 is an expression. I think it is clearer in such way. But for atan2pi we can use exact number: -0.75.

I don't have a firm opinion, I'm just pointing out a possible option. I am fine with any option.

.. as suggested in review

Branch updated based on review comments. Thank you for your reviews.

LGTM. 👍

Also use the surrounding style of two spaces after a sentence- ending period.

I've updated the PR with additional comment changes as requested.

    {"atanpi",          math_atanpi,    METH_O,       math_atanpi_doc},

make it aligned with METH_O of atanh

LGTM, with few remaining comments

Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com> Co-authored-by: Sergey B Kirpichev <skirpichev@gmail.com>

Hi. I think I have addressed all open review comments. Is there anything else I can do to help this PR move forward?

Thank you for your contribution @jepler.

Thank you for working with me on this!

…nGH-150555) Add the the following functions to the math module: acospi, asinpi, atanpi, atan2pi, cospi, sinpi, tanpi.

I created PR gh-157858 to rename HAVE_ACOSPI macro to _Py_HAVE_ACOSPI (and rename also similar macros).

jepler added 4 commits May 27, 2026 12:41

Add news item and remove testing code

03808d6

tests: Add math_testcases for the new *pi functions

e9ec2b6

fix discussion of atan2pi

bb57325

jepler requested review from AA-Turner, corona10, emmatyping and erlend-aasland as code owners May 28, 2026 14:09

bedevere-app Bot mentioned this pull request May 28, 2026

Add 'half turns' functions (cospi et al) to math module following C23 & IEEE754-2019 #150534

Closed

bedevere-app Bot added the awaiting review label May 28, 2026

jepler added 2 commits May 28, 2026 12:16

fix wrongly changed m_sinpi

7650fa5

regenerate autoconf files with make regen-configure

86de1c0

skirpichev requested changes May 29, 2026

View reviewed changes

bedevere-app Bot added awaiting core review and removed awaiting review labels May 29, 2026

skirpichev requested a review from picnixz May 29, 2026 05:14

skirpichev reviewed May 29, 2026

View reviewed changes

Comment thread Doc/library/math.rst

serhiy-storchaka reviewed May 29, 2026

View reviewed changes

jepler added 11 commits May 29, 2026 13:04

remove accidentally-committed code

be980bf

mathdata: Add +-inf, nan tests for new functions

d3254cd

test_math: Let "mtestfile" test 2-arg functions

6d8dd17

math_testcases: Add testcases for atan2pi

0c7fdfb

These are now tested in math_testcases.txt

aa93326

Add whatsnew entry for math module changes

e0ede77

Correct doc typos (atanpi2 -> atan2pi)

694f479

Correct comment about which standard this conforms to

ab68cdc

Add 'versionadded' notes to new functions

359ac19

Use float constants, not expressions, for simple values.

6cec4b8

.. as suggested in review

as x is finite, regular comparisons can be used here

d66a2c3

serhiy-storchaka approved these changes May 29, 2026

View reviewed changes

bedevere-app Bot added awaiting merge and removed awaiting core review labels May 29, 2026

skirpichev reviewed May 29, 2026

View reviewed changes

Comment thread Modules/mathmodule.c Outdated

Comment thread Modules/mathmodule.c Outdated

hugovk reviewed May 30, 2026

View reviewed changes

Comment thread Doc/library/math.rst

jepler added 3 commits May 30, 2026 10:47

remove comment about glibc

1f98ce6

Update comments based on review.

12fcd24

Also use the surrounding style of two spaces after a sentence- ending period.

math_methods: Re-sort and conform to surrounding spacing

56da6fc

jepler requested a review from skirpichev May 30, 2026 17:32

picnixz reviewed May 30, 2026

View reviewed changes

picnixz changed the title gh-150534: Add C23 math "*pi" functions gh-150534: Add C23 half-turn trigonometric *pi functions May 30, 2026

skirpichev approved these changes May 31, 2026

View reviewed changes

Comment thread Modules/mathmodule.c Outdated

Comment thread Doc/whatsnew/3.16.rst

Apply suggestions from code review

0e1ba52

Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com> Co-authored-by: Sergey B Kirpichev <skirpichev@gmail.com>

jepler requested a review from itamaro as a code owner May 31, 2026 15:18

jepler commented May 31, 2026

View reviewed changes

Comment thread Modules/mathmodule.c Outdated

jepler added 2 commits May 31, 2026 10:18

Update Modules/mathmodule.c

c763c61

apply lint fix

e243ce3

Merge branch 'main' into c23-math-pifuns

4364ecb

picnixz approved these changes Jun 6, 2026

View reviewed changes

serhiy-storchaka enabled auto-merge (squash) June 6, 2026 09:54

serhiy-storchaka merged commit b6e6613 into python:main Jun 6, 2026
56 checks passed

bedevere-app Bot removed the awaiting merge label Jun 6, 2026

fedonman mentioned this pull request Sep 7, 2026

Documentation errors in the new half-turn *pi functions #157135

Closed

                   {"atan2",           _PyCFunction_CAST(math_atan2),     METH_FASTCALL,  math_atan2_doc},
                   {"atanh",           math_atanh,     METH_O,         math_atanh_doc},
+                  {"atan2pi",         _PyCFunction_CAST(math_atan2pi),   METH_FASTCALL,  math_atan2pi_doc},
+                  {"atanpi",          math_atanpi,      METH_O,       math_atanpi_doc},

+              acospi10000 acospi 0.0 -> 0.5
+              acospi10001 acospi -0.0 -> 0.5
+              acospi10002 acospi 1.0 -> 0.0


		.. function:: atanpi(x)

		Return the arc tangent of x, in half-turns. The result is between ``-1/2`` and

gh-150534: Add C23 half-turn trigonometric `*pi` functions by jepler · Pull Request #150555 · python/cpython · GitHub

Repository navigation

gh-150534: Add C23 half-turn trigonometric *pi functions - #150555

gh-150534: Add C23 half-turn trigonometric *pi functions#150555serhiy-storchaka merged 25 commits intopython:mainpython/cpython:mainfrom jepler:c23-math-pifunsjepler/cpython:c23-math-pifunsCopy head branch name to clipboard

Conversation

jepler commented May 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

read-the-docs-community Bot commented May 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Documentation build overview

Uh oh!

skirpichev May 29, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

serhiy-storchaka left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

serhiy-storchaka May 29, 2026

Choose a reason for hiding this comment

Uh oh!

jepler May 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

serhiy-storchaka May 29, 2026

Choose a reason for hiding this comment

Uh oh!

jepler commented May 29, 2026

Uh oh!

serhiy-storchaka left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jepler commented May 30, 2026

Uh oh!

Uh oh!

picnixz May 30, 2026

Choose a reason for hiding this comment

Uh oh!

skirpichev left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jepler commented Jun 4, 2026

Uh oh!

serhiy-storchaka commented Jun 6, 2026

Uh oh!

Uh oh!

jepler commented Jun 6, 2026

Uh oh!

vstinner commented Sep 20, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

gh-150534: Add C23 half-turn trigonometric *pi functions#150555
serhiy-storchaka merged 25 commits into
python:mainfrom
jepler:c23-math-pifuns

jepler commented May 28, 2026 •
edited
Loading

read-the-docs-community Bot commented May 28, 2026 •
edited
Loading

jepler May 29, 2026 •
edited
Loading