Stefan Krah added the comment:
I have been asked to backport this to 3.8. There's a very small window
of opportunity:
3.8.6: Monday, 2020-09-21
3.8.7rc1: Monday, 2020-11-02
3.8.7: Monday, 2020-11-16 (final version!)
Backporting procedures since 3.8 are unclear and a source of
con
Change by Stefan Krah :
--
nosy: -skrah
___
Python tracker
<https://bugs.python.org/issue41732>
___
___
Python-bugs-list mailing list
Unsubscribe:
Stefan Krah added the comment:
It would have been nice to have had a chance to look at this, but
apparently those days are over.
--
___
Python tracker
<https://bugs.python.org/issue41
Stefan Krah added the comment:
list() has to *interpret* the endianness to display it. The endianness
of the *native* format does not matter. The endianness of an explicit
format matters, since list has to be able to convert non-native formats.
I suggest to look at XND, which is a memoryview
Stefan Krah added the comment:
memoryview can store anything, but only access and display
the native format. The reason is that conversions in the manner
of the struct module are very expensive.
You can construct all sorts of formats that memoryview cannot
handle:
>>> x = array(1, &
Stefan Krah added the comment:
I think this is a duplicate of #41401.
--
nosy: +skrah
resolution: -> duplicate
stage: -> resolved
status: open -> closed
superseder: -> Using non-ascii that require UTF-8 breaks AIX testing
___
Py
Change by Stefan Krah :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Stefan Krah added the comment:
New changeset fb050d0d60f38dc9b6c30df1864020a92981be5b by Miss Islington (bot)
in branch '3.9':
bpo-41721: Add xlc options (GH-22097)
https://github.com/python/cpython/commit/fb050d0d60f38dc9b6c30df1864020
Stefan Krah added the comment:
The code example is for 64-bit Linux with sizeof(long) == sizeof(long long).
It also works on 32-bit xlc with int/long.
--
___
Python tracker
<https://bugs.python.org/issue41
Stefan Krah added the comment:
New changeset 84a7917b4c9afec07575065cffa143b91fe98c14 by Stefan Krah in branch
'master':
bpo-41721: Add xlc options (GH-22096)
https://github.com/python/cpython/commit/84a7917b4c9afec07575065cffa143
Stefan Krah added the comment:
-qmaxmem=-1 is added to disable verbose remarks.
--
___
Python tracker
<https://bugs.python.org/issue41721>
___
___
Python-bug
Stefan Krah added the comment:
We would also need -fwrapv to be safe, but I cannot find an equivalent
option for xlc.
The Intel compiler had a similar failure to #40244 that was resolved
by -fwrapv (see #40223).
--
___
Python tracker
<ht
New submission from Stefan Krah :
At least the xlc version on AIX 7.1 has aggressive optimizations even
with -O:
#include
#include
static int
f(long *a, long long *b)
{
int t = *a;
*b = 0; // cannot
Change by Stefan Krah :
--
keywords: +patch
pull_requests: +21183
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/22096
___
Python tracker
<https://bugs.python.org/issu
Change by Stefan Krah :
--
nosy: David.Edelsohn, skrah
priority: normal
severity: normal
status: open
title: xlc: add -qalias=noansi -qmaxmem=-1
___
Python tracker
<https://bugs.python.org/issue41
Stefan Krah added the comment:
If nothing has been diagnosed, I suggest closing this.
--
status: open -> pending
___
Python tracker
<https://bugs.python.org/issu
Change by Stefan Krah :
--
title: Decimal divisions sometimes 10x or 100x too large -> small_ints[]
modified (third party C-extension?)
___
Python tracker
<https://bugs.python.org/issu
Stefan Krah added the comment:
This is in master and 3.9.1. I'll not backport to 3.8 because a release
candidate is imminent.
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
type: -> behavior
vers
Stefan Krah added the comment:
New changeset 88b86a9752afc2c50ca196f6ba1a8d62d71cf398 by Miss Islington (bot)
in branch '3.9':
bpo-19521: Fix parallel build race condition on AIX (GH-22001)
https://github.com/python/cpython/commit/88b86a9752afc2c50ca196f6ba1a8d
Stefan Krah added the comment:
New changeset e6dcd371b2c54a94584dd124e8c592a496d46a47 by Stefan Krah in branch
'master':
bpo-19521: Fix parallel build race condition on AIX (GH-21997)
https://github.com/python/cpython/commit/e6dcd371b2c54a94584dd124e8c592
Stefan Krah added the comment:
I can't find the reason for:
if test -z "$EXPORTSYMS"; then
EXPORTSYMS="Modules/python.exp"
fi
Modules/python.exp is hardcoded elsewhere, and I'd rather set
EXPORTSYMS unconditionally on AIX and un
Stefan Krah added the comment:
Okay, thanks. The -G option is of course attractive for Linux projects
because it requires minimal changes in the build machinery.
I've added support for libmpdec/libmpdec++ (next release) for AIX-style
shared libraries (shr.o inside libmpdec.a). Th
Change by Stefan Krah :
--
pull_requests: +21103
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/21997
___
Python tracker
<https://bugs.python.org/issu
Stefan Krah added the comment:
The original patch is a bit dated, do we still need the export symbol
generation?
https://www.ibm.com/support/knowledgecenter/en/SSGH3R_16.1.0/com.ibm.xlcpp161.aix.doc/proguide/compiling_shared_aix.html
"Use the -G option to create a shared library fro
Stefan Krah added the comment:
Well, I misunderstood this sentence then, so it's just for testing. :)
> Our customers should use it or use ulimit -d.
One will hit this issue also when following the MAX_PREC section
in the FAQ, but that is a rare case:
https://docs.python.org/3.10
Stefan Krah added the comment:
> That's probably due to a very small size of the Paging Space of the AIX
> machine you used for testing.
That is the case, the machine has 160GB of memory and 1GB of paging space. I
guess it is configured specifically for not freezing.
> About
Change by Stefan Krah :
--
nosy: +BTaskaya, Michael.Felt, kadler, skrah
___
Python tracker
<https://bugs.python.org/issue19521>
___
___
Python-bugs-list mailin
Change by Stefan Krah :
--
versions: +Python 3.10 -Python 2.7, Python 3.2, Python 3.3, Python 3.4, Python
3.5, Python 3.6, Python 3.7
___
Python tracker
<https://bugs.python.org/issue19
Stefan Krah added the comment:
I understand that both of you are in favor of #19521 (the patch of
which I have not tried yet).
Can we close this as a duplicate? Please just reopen if you disagree.
--
nosy: +skrah
resolution: -> duplicate
stage: patch review -> resolved
status
Stefan Krah added the comment:
Mark, do you think that we should document the other oddity as well
or should we close this?
--
status: open -> pending
___
Python tracker
<https://bugs.python.org/issu
Stefan Krah added the comment:
Thanks to David Edelsohn I have AIX access now. The issue reported
by Pablo is the same as #41540, for a summary see msg375480.
It is a trivial issue that requires that ulimits are in place due to
the fact that AIX over-allocates petabytes even when the physical
Change by Stefan Krah :
--
assignee: -> skrah
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python
Change by Stefan Krah :
--
assignee: -> skrah
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
type: -> compile error
___
Python tracker
<https://bugs.pyt
Stefan Krah added the comment:
New changeset 28bf82661ac9dfaf1b2d0fd0ac98fc0b31cd95bb by Miss Islington (bot)
in branch '3.9':
bpo-41540: AIX: skip test that is flaky with a default ulimit. (GH-21890)
(#21893)
https://github.com/python/cpython/commit/28bf82661ac9dfaf1b2d0fd0ac98fc
Stefan Krah added the comment:
New changeset 1864eacc22485b26c0ec0a059c9330f877861afb by Miss Islington (bot)
in branch '3.9':
bpo-40878: xlc cannot handle C99 extern inline. (GH-21891)
https://github.com/python/cpython/commit/1864eacc22485b26c0ec0a059c9330
Stefan Krah added the comment:
New changeset 39dab24621122338d01c1219bb0acc46ba9c9956 by Stefan Krah in branch
'master':
bpo-41540: AIX: skip test that is flaky with a default ulimit. (#21890)
https://github.com/python/cpython/commit/39dab24621122338d01c1219bb0acc
Change by Stefan Krah :
--
keywords: +patch
pull_requests: +21009
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/21890
___
Python tracker
<https://bugs.python.org/issu
Stefan Krah added the comment:
To recap for people who find this: The problem occurs because of AIX's
extreme over-allocation and is specific to the 64-bit build.
Workarounds:
1) Something like ulimit -d 800.
2) xlc: LDFLAGS="-L/usr/lib64 -q64 -bmaxdata:0x8"
Stefan Krah added the comment:
> -qmaxmem affects the compiler optimization.
I know, that's just from the Python README.AIX. I didn't expect it to have any
influence.
> -Wl,-bmaxdata:0x8 is a GCC command line option.
That is indeed surprising. Linking is prep
Stefan Krah added the comment:
New changeset 40e700ad042089120456cc2ee79b8ca69479416b by Stefan Krah in branch
'master':
bpo-40878: xlc cannot handle C99 extern inline. (GH-21887)
https://github.com/python/cpython/commit/40e700ad042089120456cc2ee79b8c
Change by Stefan Krah :
--
keywords: +patch
pull_requests: +21006
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/21887
___
Python tracker
<https://bugs.python.org/issu
Stefan Krah added the comment:
Thank you, David!
Now that I can test on AIX, I can confirm that the data limit is the
culprit:
libmpdec deliberately calls malloc(52631578947368422ULL) in the
maxprec tests, which is supposed to fail, but succeeds.
However, instead of freezing the machine
Stefan Krah added the comment:
> So, this issue with v3.10 (master) appeared to me as a regression.
I understand that from your point of view it appears as a regression.
However, quoting the C standard, 7.20.3 Memory management functions:
"The pointer returned points to the start
Stefan Krah added the comment:
> Core developers have full access to AIX system for the asking. Back to you,
> Stefan.
That sounds great. Can we contact you directly, or have I missed an earlier
announcement from someone else giving out AIX access?
Or are you working on it
Stefan Krah added the comment:
The test (size > (size_t)PY_SSIZE_T_MAX)) has nothing to do with it. Within
Python, most sizes are ssize_t, so a value larger than SSIZE_MAX is suspicious.
AIX is an unsupported platform.
Realistically, if people want AIX to be supported, someone has to g
Stefan Krah added the comment:
Also, perhaps build Python with -bmaxdata?
https://www.enterprisedb.com/edb-docs/d/postgresql/reference/manual/11.1/installation-platform-notes.html
--
___
Python tracker
<https://bugs.python.org/issue41
Stefan Krah added the comment:
We need more information.
Is this 64-bit AIX?
How much physical memory does the machine have?
Linux also has over-allocation and the default for ulimit is unlimited.
But it does not attempt to over-allocate such an outrageous amount of
memory.
Neither do
Change by Stefan Krah :
--
components: +C API -Extension Modules
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Stefan Krah added the comment:
New changeset 39042e00ab01d6521548c1b7cc6554c09f4389ff by Stefan Krah in branch
'master':
bpo-41324 Add a minimal decimal capsule API (#21519)
https://github.com/python/cpython/commit/39042e00ab01d6521548c1b7cc6554
Stefan Krah added the comment:
New changeset 9b9f1582753979f38d2fd927cddf0621a65e9ed6 by Stefan Krah in branch
'master':
bpo-41369 Update to libmpdec-2.5.1: new features (GH-21593)
https://github.com/python/cpython/commit/9b9f1582753979f38d2fd927cddf06
Change by Stefan Krah :
--
keywords: +patch
pull_requests: +20732
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/21593
___
Python tracker
<https://bugs.python.org/issu
New submission from Stefan Krah :
This issue tracks the update of the included libmpdec to version 2.5.1.
The version includes features required for #41324.
--
assignee: skrah
components: Extension Modules
messages: 374101
nosy: skrah
priority: normal
severity: normal
status: open
Stefan Krah added the comment:
I'm happy with the API, except that --with-system-libmpdec is naturally
broken. So I've to release a new libmpdec, which I'd rather do soon
because I don't want to revisit this later.
Antoine has looked at the API. If anyone else has re
Stefan Krah added the comment:
Thanks, I can confirm that the speedup for memoryview.cast() is large in
microbenchmarks.
Still, if the speedup cannot be seen in programs like test_buffer,
the amount of code that's being added for a speedup < 5% in real
world programs is quite substan
Stefan Krah added the comment:
New changeset 10e466448f67850ed7bb2e2a4e7f017f2b050cad by Srinivas Reddy
Thatiparthy (శ్రీనివాస్ రెడ్డి తాటిపర్తి) in branch 'master':
bpo-41205: Document Decimal power 0 to the 0 (GH-21386)
https://github.com/python/cpyt
Stefan Krah added the comment:
It looks like the API would be usable, so the PR now has documentation.
--
___
Python tracker
<https://bugs.python.org/issue41
Stefan Krah added the comment:
I cannot detect a speedup in test_buffer, which is a heavy user of memoryviews:
# before:
>>> a = [3.742, 3.589, 3.542, 3.495, 3.481, 3.620, 3.773, 3.755, 3.701, 3.661]
>>> sum(a) / 10
3.63589995
# after
>>> b = [3.63, 3.59
Stefan Krah added the comment:
Adding Daniele Varrazzo, in case this is useful for the PostgreSQL
adapter.
--
nosy: +piro
___
Python tracker
<https://bugs.python.org/issue41
Stefan Krah added the comment:
Also as a note for Mark and Raymond: This API is for exact conversions
and avoids the use of the context except for raising ConversionSyntax.
I'll add documentation once Antoine has tried it out for his use
Change by Stefan Krah :
--
type: -> enhancement
___
Python tracker
<https://bugs.python.org/issue41324>
___
___
Python-bugs-list mailing list
Unsubscrib
Stefan Krah added the comment:
Closing in favor of #41324, which adds just the most important
functions.
--
resolution: -> duplicate
stage: -> resolved
status: open -> closed
superseder: -> Add a minimal decimal capsule API
___
Py
New submission from Stefan Krah :
This adds a minimal decimal capsule API. As can be seen from the patch,
adding anything to decimal while doing it properly is quite labor and
testing intensive, so the intent is to *keep* the API minimal!
That said, some functions are really necessary:
1
Change by Stefan Krah :
--
keywords: +patch
pull_requests: +20656
stage: needs patch -> patch review
pull_request: https://github.com/python/cpython/pull/21519
___
Python tracker
<https://bugs.python.org/issu
Change by Stefan Krah :
--
assignee: skrah
components: Extension Modules
nosy: skrah
priority: normal
severity: normal
stage: needs patch
status: open
title: Add a minimal decimal capsule API
versions: Python 3.10
___
Python tracker
<ht
Stefan Krah added the comment:
The top level decimal.py that dispatches to either _decimal or
_pydecimal is pure Python. So perhaps these applications could
add these methods themselves:
>>> import decimal
>>> def exp(x):
... return x.exp()
...
>>> deci
Stefan Krah added the comment:
Mark has already mentioned that setting the recursion limit to a
value higher than 2000-1 makes no sense.
Apart from that, sys.maxsize is actually documented like this:
maxsize -- the largest supported length of containers.
So it applies to Py_ssize_t
Stefan Krah added the comment:
Closing, thanks for all the patches!
--
stage: -> resolved
status: open -> closed
versions: +Python 3.8
___
Python tracker
<https://bugs.python.org/i
Stefan Krah added the comment:
New changeset 16eea45fbd3b7c3d1b222b7eb7a5d7ee427f70bd by Felix Yan in branch
'3.8':
[3.8] bpo-41302: Support system libmpdec 2.5 for Python 3.8 (GH-21488)
https://github.com/python/cpython/commit/16eea45fbd3b7c3d1b222b7eb7a5d7
Stefan Krah added the comment:
I'm going to reclassify this as a build fix for 3.8. 3.8 promises:
#if !defined(MPD_VERSION_HEX) || MPD_VERSION_HEX < 0x02040100
#error "libmpdec version >= 2.4.1 required"
#endif
So it seems reasonable to support at least two or thr
Change by Stefan Krah :
--
resolution: -> duplicate
stage: -> resolved
status: open -> closed
superseder: -> Allow objects implemented in pure Python to export PEP 3118
buffers
___
Python tracker
<https://bugs.python
Stefan Krah added the comment:
Yes, let's make #13797 a superseder.
--
___
Python tracker
<https://bugs.python.org/issue41285>
___
___
Python-bugs-list m
Stefan Krah added the comment:
If I understand this correctly, I think this may be a duplicate of #13797.
Something like a __getbuffer__ protocol would be needed. I'm not
saying we *should* go that route, just stating the requirements
from my perspe
Change by Stefan Krah :
--
resolution: -> fixed
stage: patch review ->
status: open -> pending
___
Python tracker
<https://bugs.python.org/issue41302>
___
___
Stefan Krah added the comment:
Thanks for the patch!
The integrated libmpdec-2.4.2 in Python 3.8 still has a couple of
UNUSED, so the simple approach of moving the define would not work.
It would be easy to replace the few instances of UNUSED with void
casts like in 2.5.0.
Strictly speaking
Stefan Krah added the comment:
New changeset 015efdbef7454a522e88cd79ba2b4cd77a5fb2a2 by Felix Yan in branch
'master':
bpo-41302: Fix build with system libmpdec (GH-21481)
https://github.com/python/cpython/commit/015efdbef7454a522e88cd79ba2b4c
Stefan Krah added the comment:
Thank you for the report, I'll add the define or remove UNUSED in 3.9
and 3.10.
3.8 is still supposed to use libmpdec-2.4.2, though it would be
harmless to use libmpdec-2.5.0.
Are you planning to use libmpdec-2.5.0 with 3.8?
--
assignee: -&g
Stefan Krah added the comment:
s/PyPy as a bottleneck/the C-API as a bottle neck for PyPy/
--
___
Python tracker
<https://bugs.python.org/issue39542>
___
___
Stefan Krah added the comment:
A brief note for Victor that *nothing* was directed against him. On
the contrary, msg372995 was supporting him in case the commit had
actually been unreviewed, which it apparently wasn't. Sorry for
jumping on the bandwagon there.
> PEP 620 for the over
Stefan Krah added the comment:
More than that, I had *promised* Matthias privately to release a
new libmpdec for the sqrt-max-prec feature a couple of months ago.
I request that further packaging issues will be dealt with primarily
by Matthias and myself
Stefan Krah added the comment:
> The standalone libmpdec had to be updated, and was updated according
to the Debian-friendly way requested by Matthias himself.
Not updated, of course the sqrt-max-prec *had never been* in
the standalone libmpdec, it is new in 2.
Stefan Krah added the comment:
As noted in the first message of this thread, the sqrt-max-prec feature
(requested by Mark and Tim) was already in 3.9 long before the beta
freeze.
I'm not sure why this is not clear from the original message.
That fix is safe for Python, but not fo
Change by Stefan Krah :
--
nosy: +skrah
___
Python tracker
<https://bugs.python.org/issue41215>
___
___
Python-bugs-list mailing list
Unsubscribe:
Stefan Krah added the comment:
Thanks for taking a look, Antoine.
IIRC, the version pinning already accommodates Debian:
#if !defined(MPD_VERSION_HEX) || MPD_VERSION_HEX < 0x0205
#error "libmpdec version >= 2.5.0 required"
#endif
In the first libmpdec versions,
Stefan Krah added the comment:
> In any case, I would have had a hard time giving a competent opinion on this
> issue.
Essentially it's a really simple Linux packaging issue for the
external libmpdec. To have the exact same behavior for the external
libmpdec as for the includ
Stefan Krah added the comment:
I have no opinion about this commit (I did not benchmark anything,
but I wouldn't be surprised if Victor did).
I do have the opinion that unreviewed commits occur quite frequently
and nearly every committer has made some (or a lot), including the
committer
Stefan Krah added the comment:
Agreed, and it's even slightly worse with the ROUND_FLOOR special case:
>>> c.rounding = ROUND_FLOOR
>>> +Decimal("-0")
Decimal('-0')
>>>
That's why there are three slightly different method
Stefan Krah added the comment:
Ah, I see that the subject has already been answered in the negative:
https://discuss.python.org/t/make-code-review-mandatory/4174
He must have forgotten.
--
___
Python tracker
<https://bugs.python.org/issue39
Stefan Krah added the comment:
Christian Heimes has expressed an interest (quite rudely) in unreviewed commits
elsewhere. I wonder if that applies to everyone regardless of employer.
--
nosy: +christian.heimes, skrah
___
Python tracker
<ht
Stefan Krah added the comment:
How witty!
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
type: -> behavior
___
Python tracker
<https://bugs.python
Stefan Krah added the comment:
> Finally, while Raymond and Antoine are welcome to voice their opinions on the
> matter, your change is landing in 3.9.0b4 which I'm about to announce. So we
> won't be reverting it. In the future let's make sure we stick to the rele
Stefan Krah added the comment:
Finally, about the Debian issue, of course you could also link 3.8
against the static lib.
--
___
Python tracker
<https://bugs.python.org/issue40
Stefan Krah added the comment:
Łukasz, which one is nicer?
> reverting this patch passes all the tests, what's the motivation and why were
> there no code reviews for this?
or:
> Yeah, I already felt a bit guilty about adding you -- it could be a compiler
> bug or an act
Stefan Krah added the comment:
> both 3.8 and 3.9 have to be available on the systems for the transition
> period.
If sonames can be incremented for libraries even if they are ABI compatible,
how about using up as many as you need for the Debian
package? Next time when I release mpd
Stefan Krah added the comment:
> In the mean time may I request that you follow our protocol for code review.
Ah, who reviews libffi? It is just updated. Not counting the thousands of
other cases people commit unreviewed, like in changing the C-API.
And no, Christian, that is
Stefan Krah added the comment:
Matthias, to tell the truth, I was never sure about the soname.
I read this:
https://www.debian.org/doc/debian-policy/ch-sharedlibs.html
"""
The SONAME and binary package name need not, and indeed normally should not,
change if new interface
Stefan Krah added the comment:
Since this issue has been brought to the attention of the CoC
committee. Here is how *I* report issues:
https://bugs.python.org/issue40223#msg372578
https://bugs.python.org/issue40223#msg372637
https://github.com/google/sanitizers/issues/1257
Note
Stefan Krah added the comment:
> backwards incompatible changes.
It is not a backwards incompatible change. Slamming a 3.9 into
a nightly build has never been supported.
--
___
Python tracker
<https://bugs.python.org/issu
Stefan Krah added the comment:
Raymond, Mark, Antoine: If you think this should be reverted, I'll
revert it.
--
nosy: +mark.dickinson, pitrou, rhettinger
___
Python tracker
<https://bugs.python.org/is
Stefan Krah added the comment:
I was accused of breaking the release, which is false. It is outside
of a release manager's authority to claim that an *experimental* and
*nightly* build that uses a flag in an unintended manner needs counts
as bre
Change by Stefan Krah :
--
components: +Extension Modules
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
type: -> behavior
___
Python tracker
<https://bugs.python
1 - 100 of 3396 matches
Mail list logo