You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
test.test_math.MathTests.testAtan2 failure on OpenIndiana/Solaris #138573
changed the title [-]`test.test_math.MathTests.testAtan2` failure on OpenIndiana[/-][+]`test.test_math.MathTests.testAtan2` failure on OpenIndiana/Solaris[/+]on Sep 6, 2025
I'm interested by test_cext and test_cppext failures, but also by test_embed failure. Would you mind to open an issue for test_cext+test_cppext and another one for test_embed?
I'm interested by test_cext and test_cppext failures, but also by test_embed failure. Would you mind to open an issue for test_cext+test_cppext and another one for test_embed?
Activity
StanFromIreland commentedon Sep 6, 2025
Also occurs on our Solaris buildbot.
I actually talked about this with @kulikjak, and this is an issue on their side. So there isn't anything for us to do.
[-]`test.test_math.MathTests.testAtan2` failure on OpenIndiana[/-][+]`test.test_math.MathTests.testAtan2` failure on OpenIndiana/Solaris[/+]terryjreedy commentedon Sep 6, 2025
Does that mean we should close this?
skirpichev commentedon Sep 6, 2025
math.atan2 is essentially a thin wrapper over libm's atan2:
cpython/Modules/mathmodule.c
Lines 1100 to 1103 in 8554c09
cpython/Modules/mathmodule.c
Lines 1014 to 1047 in 8554c09
I don't think something was changed here since v3.13 other than using libm's isinf/isnan macro directly.
It looks like in the given case platform's atan2 returns a nan. @mtelka, could you verify this?
Correct answer is pi, see https://en.cppreference.com/w/c/numeric/math/atan2.html (Assuming IEEE 754-compatible behavior.) @mtelka, did you test on same version of the OS? This looks as a regression for me, but not in the CPython.
Edit: I assume, that we won't add a workaround for unsupported broken platform. Or?
StanFromIreland commentedon Sep 6, 2025
Yes, as I said, this is an issue in their library, which they hope to fix soon.
skirpichev commentedon Sep 6, 2025
Ok, let's close. I don't think workaround is needed.
mtelka commentedon Sep 6, 2025
I tested this simple C:
and here is the result:
How can I prove the issue is with the platform?
32 remaining items
pythongh-138573: filter out failing math tests on Solaris
skirpichev commentedon Mar 25, 2026
@mtelka, see #146402.
vstinner commentedon Mar 25, 2026
Wow, this is an old bug! 😄
I'm interested by test_cext and test_cppext failures, but also by test_embed failure. Would you mind to open an issue for test_cext+test_cppext and another one for test_embed?
gh-138573: Filter out failing math tests on Solaris (#146402)
pythongh-138573: Filter out failing math tests on Solaris (pythonGH-1…
vstinner commentedon Mar 25, 2026
The two failing tests are now skipped on Solaris: 8e1469c. The change will be backported to 3.14 once the CI pass on the PR.
Thanks for your bug report @mtelka.
vstinner commentedon Mar 25, 2026
Oh, this issue has been fixed by the change illumos/illumos-gate@2eba8eb in the meanwhile, nice!
What is
lib_version != libm_ieeetest in the implementation? It's possible to select a "lib_version" to get different behaviors?[3.14] gh-138573: Filter out failing math tests on Solaris (GH-146402) (
mtelka commentedon Mar 26, 2026
I do not think so. However, there was a discussion about this line during the code review. You can find it here: https://code.illumos.org/c/illumos-gate/+/4476
mtelka commentedon Mar 26, 2026
These tests pass with Python 3.14.3.
kulikjak commentedon Mar 31, 2026
Thank you for the merge. We still didn't fix (or change) the
atan2behavior, so the skip at least helps us.pythongh-138573: Filter out failing math tests on Solaris (python#146402
pythongh-138573: Filter out failing math tests on Solaris (python#146402