Filip Gruszczyński grusz...@gmail.com added the comment:
Pleasure :-)
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13544
___
___
Ezio Melotti ezio.melo...@gmail.com added the comment:
Terry, does the latest patch look good to you?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6570
___
Antoine Pitrou pit...@free.fr added the comment:
Very high-level comments on your patch:
- why an empty dtrace module?
- I'm worried that you're adding lots of delicate code inside critical core
functions. Perhaps most of it can be factored out in separate functions living
in another
Antoine Pitrou pit...@free.fr added the comment:
I'm not convinced by the decorator approach. Why not simply add with
self.lock at the beginning of each protected method? It would actually save a
function call indirection.
--
stage: - patch review
versions: +Python 3.3 -Python 3.2
Charles-François Natali neolo...@free.fr added the comment:
Alan, I would open a new issue tracking this one and posting a patch there,
if I were you.
Previous DTRACE attempts failed because trying to make everybody happy. I
don't want to repeat the mistake.
Sorry, but I'm -1.
I don't
Giampaolo Rodola' g.rod...@gmail.com added the comment:
Are you suggesting to enable thread-synchronization by default and get rid of
explicit synchronized argument?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8684
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
nosy: +rprosser
versions: -Python 3.1
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue7867
___
Antoine Pitrou pit...@free.fr added the comment:
Are you suggesting to enable thread-synchronization by default and get
rid of explicit synchronized argument?
That would be even better indeed, if you find out that there's no
significant performance degradation.
--
anatoly techtonik techto...@gmail.com added the comment:
2011/12/12 Charles-François Natali rep...@bugs.python.org
Charles-François Natali neolo...@free.fr added the comment:
Alan, I would open a new issue tracking this one and posting a patch
there, if I were you.
Previous DTRACE
Antoine Pitrou pit...@free.fr added the comment:
Ok, a couple of further (minor) issues:
- I don't think AssertionError is the right exception type. TypeError should be
used when a type mismatches (e.g. not an unicode object);
- you don't need to check for d_type being NULL, since other methods
Giampaolo Rodola' g.rod...@gmail.com added the comment:
This is what I get by using bench.py script attached to issue13451:
CURRENT VERSION (NO LOCK)
test_cancel: time=0.67457 : calls=1 : stdev=0.0
test_empty : time=0.00025 : calls=1 : stdev=0.0
Antoine Pitrou pit...@free.fr added the comment:
Versions #2 and #3 have the same cost, so it's better to get rid of
the explicit argument and always use the lock (version #3).
Differences between #1 and #3 suggest that introducing the lock
obviously have a cost though.
It's not too high
kxroberto kxrobe...@users.sourceforge.net added the comment:
(I often wonder why software today isn't much faster than years ago -
though the nominal speed of hardware increases tremendously. package
sizes grow, without appropriate growth of functionality. This is one
example how the
Giampaolo Rodola' g.rod...@gmail.com added the comment:
New patch in attachment. I'll commit it later today.
--
nosy: +rhettinger
Added file: http://bugs.python.org/file23925/sched-thread-safe.patch
___
Python tracker rep...@bugs.python.org
sbt shibt...@gmail.com added the comment:
Patch which add __qualname__ to builtin_function_or_method. Note that I had to
make a builtin staticmethod have __self__ be the type instead of None.
--
Added file: http://bugs.python.org/file23926/method_qualname.patch
Giampaolo Rodola' g.rod...@gmail.com added the comment:
Thread locks introduced in issue8684 should make this change more robust.
If this patch is reasonable, I'd like to commit it before the one in issue8684
for simplicity.
Raymond?
--
___
Python
sbt shibt...@gmail.com added the comment:
Ok, a couple of further (minor) issues:
- I don't think AssertionError is the right exception type. TypeError
should be used when a type mismatches (e.g. not an unicode object);
- you don't need to check for d_type being NULL, since other methods
Changes by Berker Peksag berker.pek...@gmail.com:
--
nosy: +berkerpeksag
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9856
___
___
Antoine Pitrou pit...@free.fr added the comment:
Overall I wonder why you post here on the main topic resource usage,
when you don't care about issues of magnitude 2x memory usage.
Because you are talking about a fixed overhead of a mere 2MB (IIUC),
which is moreover shared between all
New submission from Bithin A bithin2...@gmail.com:
In the documentation 'HOWTO Fetch Internet Resources Using urllib2' there is a
correction under the heading 'Basic Authentication'
In the line 'The header looks like : ``Www-authenticate: SCHEME
realm=REALM``. ' the word 'Www-authenticate'
Changes by Florent Xicluna florent.xicl...@gmail.com:
--
dependencies: +Change object.__format__(s) where s is non-empty to a TypeError
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13248
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 24238e89f938 by Antoine Pitrou in branch 'default':
Issue #13577: various kinds of descriptors now have a __qualname__ attribute.
http://hg.python.org/cpython/rev/24238e89f938
--
nosy: +python-dev
Antoine Pitrou pit...@free.fr added the comment:
I think most of these shouldn't be removed, as it would make porting from 2.x
significantly more tedious.
Things that I think can be removed:
- max_buffer_size (io is new and that argument was never used by anybody)
- get_prefix / set_prefix
psam pk.sam...@gmail.com added the comment:
The problem was detected in a Django project, as the template engine was not
able to support the original encoding.
I don't have a real test code snippet, but you may try something like:
Changes by STINNER Victor victor.stin...@haypocalc.com:
--
nosy: +haypo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13539
___
___
Antoine Pitrou pit...@free.fr added the comment:
About method_qualname.patch:
- apparently you forgot to add BuiltinFunctionPropertiesTest in test_main()?
- a static method keeps a reference to the type: I think it's ok, although I'm
not sure about the consequences (Guido, would you have an
Philipp Lies p...@bethgelab.org added the comment:
a) it's 122GB free RAM (out of 128GB total RAM)
b) when I convert the numpy array to a list it works. So seems to be a problem
with cPickle and numpy at/from a certain array size
c) $ /usr/bin/time -v python test_np.py
Traceback (most recent
Changes by Florent Xicluna florent.xicl...@gmail.com:
--
dependencies: -Change object.__format__(s) where s is non-empty to a TypeError
Added file: http://bugs.python.org/file23928/issue13248_argparse_io_lib2to3.diff
___
Python tracker
Changes by Florent Xicluna florent.xicl...@gmail.com:
Removed file: http://bugs.python.org/file23905/issue13248_obsolescence.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13248
___
Florent Xicluna florent.xicl...@gmail.com added the comment:
I removed object.__format__ from the patch because it is not planned for
removal in 3.3, but in 3.4 (see issue #9856).
I've fixed the documentation *versionchanged* and split the patch in two.
The first is less controversial.
The
sbt shibt...@gmail.com added the comment:
- apparently you forgot to add BuiltinFunctionPropertiesTest in
test_main()?
Yes. Fixed.
- a static method keeps a reference to the type: I think it's ok, although
I'm not sure about the consequences (Guido, would you have an idea?)
Nikolaus Rath nikol...@rath.org added the comment:
On 12/11/2011 10:17 PM, Raymond Hettinger wrote:
Raymond Hettinger raymond.hettin...@gmail.com added the comment:
'''
Example code:
with CleanupManager() als mngr:
tmpdir = tempfile.mkdtemp()
Jesús Cea Avión j...@jcea.es added the comment:
According to Stan Cox, this patch almost work with SystemTap. Moreover, there
are work porting DTrace to Linux.
DTrace helping to improve performance is secondary. The real important thing is
the observability. It is difficult to understand the
Roger Serwy roger.se...@gmail.com added the comment:
I rarely have IDLE crash on Linux. If you're experiencing these issues on
Windows, see #13582.
--
nosy: +serwy
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13571
Amaury Forgeot d'Arc amaur...@gmail.com added the comment:
I just checked the file python2.6_2.6.6-8.diff.gz from the Debian python2.6
package: http://packages.debian.org/squeeze/python2.6
This diff file contains a patch to build _hashlib and _ssl extensions
statically that modifies
Antoine Pitrou pit...@free.fr added the comment:
- Interactive interpreter invocation: 517 blocks, 95128 bytes.
Note that http://bugs.python.org/issue13390 also proposes to count
allocations in the interpreter.
--
___
Python tracker
anatoly techtonik techto...@gmail.com added the comment:
How different is the performance cost of this solution compared to inserting
DTrace probe for the same purpose?
--
nosy: +techtonik
___
Python tracker rep...@bugs.python.org
STINNER Victor victor.stin...@haypocalc.com added the comment:
How different is the performance cost of this solution compared
to inserting DTrace probe for the same purpose?
DTrace is only available on some platforms (Solaris and maybe FreeBSD?).
--
Jesús Cea Avión j...@jcea.es added the comment:
- why an empty dtrace module?
This is preliminary. I am thinking about dynamic probes, something like
logging module but using dtrace. Still experimenting, not sure is actually
possible.
Martin V Löwis suggested to use sys.flags.
Undecided
Changes by Jesús Cea Avión j...@jcea.es:
--
nosy: +jcea
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13390
___
___
Python-bugs-list mailing list
anatoly techtonik techto...@gmail.com added the comment:
On Mon, Dec 12, 2011 at 6:11 PM, STINNER Victor rep...@bugs.python.orgwrote:
STINNER Victor victor.stin...@haypocalc.com added the comment:
How different is the performance cost of this solution compared
to inserting DTrace probe
Jesús Cea Avión j...@jcea.es added the comment:
- Interactive interpreter invocation: 517 blocks, 95128 bytes.
Note that http://bugs.python.org/issue13390 also proposes to count
allocations in the interpreter.
The thing is, I get this data WITHOUT touching python interpreter, using a
Antoine Pitrou pit...@free.fr added the comment:
Notice too, that the data I am showing is the extra memory I am using
for the dtrace stack helper, not all python memory (if you check the
dtrace script, I only contabilize PyMem_Malloc() when called from
PyCode_New()).
DTrace allows me to
Brett Cannon br...@python.org added the comment:
This fails for me on OS X Snow Leopard using LLVM 3.0.
And I agree with your initial guess, Victor: I don't see how importlib could
possibly be the issue here since it's using load_dynamic() and not loading some
Python source itself.
New submission from Brett Cannon br...@python.org:
The internal closure functions (eg. wrapper functions used by decorators)
should not use generic names like inner() or wrapper(), but descriptive names
so that they make sense when read in a traceback. IOW, you shouldn't have to
look up the
Amaury Forgeot d'Arc amaur...@gmail.com added the comment:
terminate abruptly? I thought that print(file=None) silently returned,
without printing but without an error.
A delayed popup to display (otherwise discarded) output is a nice feature,
though.
--
nosy: +amaury.forgeotdarc
Roger Serwy roger.se...@gmail.com added the comment:
You can try triggering bug #8900 quite simply. From a shell or an editor, press
Ctrl+N and then Ctrl+O. Open a file and watch IDLE terminate abruptly.
Also, see #12274.
If you want to play with this problem further, try adding a raise
anatoly techtonik techto...@gmail.com added the comment:
On Mon, Dec 12, 2011 at 6:26 PM, Antoine Pitrou rep...@bugs.python.orgwrote:
I am not disputing the flexibility of dtrace. However, it is also
platform-specific, and needs you to learn a dedicated programming
language and API.
If it
Charles-François Natali neolo...@free.fr added the comment:
@Antoine
Couldn't this be linked to #11564 (pickle not 64-bit ready)? AFAICT this wasn't
fixed in 2.7. Basically, an integer overflow, and malloc() would bail out when
asked a ridiculous size.
@Philipp
I'd be curious to see the last
Éric Araujo mer...@netwok.org added the comment:
Given the discussions we’ve had this month and a few months earlier about
people porting directly to 3.2 or 3.3, I think the second patch needs to wait
for 3.4 or 3.5. (I have no opinion on the first patch, not being a customer of
lib2to3 or
Charles-François Natali neolo...@free.fr added the comment:
@Antoine
Couldn't this be linked to #11564 (pickle not 64-bit ready)? Basically, an
integer overflow, and malloc() would bail out when asked a ridiculous size.
AFAICT this wasn't fixed in 2.7.
@Philipp
I'd be curious to see the last
Changes by Charles-François Natali neolo...@free.fr:
--
Removed message: http://bugs.python.org/msg149321
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13555
___
Ezio Melotti ezio.melo...@gmail.com added the comment:
I think it would be better to decide in what version each thing will be removed
and then mark them with the deprecated-removed directive in the doc.
--
___
Python tracker rep...@bugs.python.org
Éric Araujo mer...@netwok.org added the comment:
I don’t know this module well, so I’m adding Georg, who last changed that
method, and other people from #10092 to the nosy list.
--
nosy: +JJeffries, Retro, christian.heimes, georg.brandl, ixokai,
r.david.murray, tim.golden, twouters
Éric Araujo mer...@netwok.org added the comment:
Thanks for the idea Jesús, even though I didn’t get the change to use it :)
Filip: Is there a reasons for using a nonlocal count rather than e.g.
self.count? Otherwise the test looks good.
--
___
Éric Araujo mer...@netwok.org added the comment:
Lars, as part of a small doc patch I want to change this in tarfile.rst:
The :mod:`tarfile` module makes it possible to read and write tar
archives, including those using gzip or bz2 compression.
-(:file:`.zip` files can be read and
Antoine Pitrou pit...@free.fr added the comment:
Couldn't this be linked to #11564 (pickle not 64-bit ready)?
Well, I don't know anything about numpy, but:
196 * 24
4704
196 * 24 * 8 # assuming 8 bytes per float
37632
2**31
2147483648
So it seems unlikely to be the
Changes by Berker Peksag berker.pek...@gmail.com:
--
nosy: +berkerpeksag
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13588
___
___
Antoine Pitrou pit...@free.fr added the comment:
Oh, what's more interesting is that it works here (Python 2.7.1 and numpy 1.6.1
under Mageia with 8GB RAM). Looking at a pickle disassembly, the only
remarkable thing is the presence of a long binary string (the raw serialization
of all IEEE
Changes by Berker Peksag berker.pek...@gmail.com:
--
nosy: +berkerpeksag
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11175
___
___
Changes by Berker Peksag berker.pek...@gmail.com:
--
nosy: +berkerpeksag
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11468
___
___
Antoine Pitrou pit...@free.fr added the comment:
Filip: Is there a reasons for using a nonlocal count rather than e.g.
self.count? Otherwise the test looks good.
Eric, please, could we stop such pointless nitpicking about one's
stylistic preferences? nonlocal is as good as any other solution
Filip Gruszczyński grusz...@gmail.com added the comment:
I haven't given it much thought, when I was making the choice of using nonlocal
rather than self.count. I was rather excited to see, if the change will work as
I wanted it to. If you believe it would be better to use an attribute, I'll
Jesús Cea Avión j...@jcea.es added the comment:
Well, if this is going to be applied to 2.7, nonlocal is invalid syntax
there. I guess that being able to use the same test in 2.7 and 3.2/3.3 would be
nice. Style, but justified.
--
___
Python
Lars Gustäbel l...@gustaebel.de added the comment:
Please, go ahead!
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5689
___
___
Changes by Eric Snow ericsnowcurren...@gmail.com:
--
nosy: +eric.snow
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13588
___
___
Python-bugs-list
Antoine Pitrou pit...@free.fr added the comment:
Well, if this is going to be applied to 2.7, nonlocal is invalid
syntax there.
Ah, good point.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13521
Filip Gruszczyński grusz...@gmail.com added the comment:
I'll send a patch, when I get home from work.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13521
___
Raymond Hettinger raymond.hettin...@gmail.com added the comment:
I'll be updating this example shortly, but it is intentional that it include
only assertEqual, assertTrue, and assertRaises. Those three are the minimum
necessary to get up and running (which is the whole point of the BASIC
Raymond Hettinger raymond.hettin...@gmail.com added the comment:
Let's just start with a working 2.7 patch and go from there.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13521
___
Antoine Pitrou pit...@free.fr added the comment:
I suppose the status can be switched to closed?
--
nosy: +pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13544
___
Antoine Pitrou pit...@free.fr added the comment:
+1.
--
nosy: +pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13575
___
___
Python-bugs-list
Antoine Pitrou pit...@free.fr added the comment:
Only worry is that codecs.latin_1_encode.__module__ is '_codecs', and
_codecs is undocumented.
It seems we have to choose between two evils here. Given that the
codecs.latin_1_encode produces more compact pickles, I'd say go for it.
Note that
Changes by Oleg Plakhotnyuk oleg...@gmail.com:
Removed file: http://bugs.python.org/file23734/test_aifc.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13394
___
Meador Inge mead...@gmail.com added the comment:
Yup, oversight on my part. Thanks.
--
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13544
___
Oleg Plakhotnyuk oleg...@gmail.com added the comment:
Sounds perfectly reasonable. Here goes the first patch with pure test coverage.
--
Added file: http://bugs.python.org/file23931/test_aifc.patch
___
Python tracker rep...@bugs.python.org
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 021e5bb297d1 by Florent Xicluna in branch 'default':
Issue #13575: there is only one class type.
http://hg.python.org/cpython/rev/021e5bb297d1
--
nosy: +python-dev
___
Changes by Florent Xicluna florent.xicl...@gmail.com:
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13575
Ezio Melotti ezio.melo...@gmail.com added the comment:
The patch includes only assertEqual, assertTrue, and assertRaises and, except a
s/functions/methods/ in the first line, looks good to me.
--
stage: needs patch - commit review
versions: -Python 3.1
New submission from Oleg Plakhotnyuk oleg...@gmail.com:
1. Fixed _write_float to handle infinity, NaN and negative numbers correctly.
2. Renamed _write_long to _write_ulong because it actually writes unsigned
value.
3. Added _read_ushort as counterpart of _write_ushort. This is never used
Oleg Plakhotnyuk oleg...@gmail.com added the comment:
Patch must be applied after http://bugs.python.org/file23931/test_aifc.patch
from issue 13394.
Not sure if review tool can handle this correctly.
--
___
Python tracker rep...@bugs.python.org
Oleg Plakhotnyuk oleg...@gmail.com added the comment:
Second patch goes to issue 13589
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13589
___
Oleg Plakhotnyuk oleg...@gmail.com added the comment:
Second patch goes to issue 13589
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13394
___
Changes by Oleg Plakhotnyuk oleg...@gmail.com:
--
type: - behavior
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13589
___
___
Python-bugs-list
Raymond Hettinger raymond.hettin...@gmail.com added the comment:
Ezio, please leave this one for me.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11468
___
Changes by Nikolaus Rath nikol...@rath.org:
Added file: http://bugs.python.org/file23933/CleanupManager_patch_v2.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13585
___
Changes by Oleg Plakhotnyuk oleg...@gmail.com:
Removed file: http://bugs.python.org/file23931/test_aifc.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13394
___
Changes by Oleg Plakhotnyuk oleg...@gmail.com:
Added file: http://bugs.python.org/file23934/test_aifc.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13394
___
Changes by Giampaolo Rodola' g.rod...@gmail.com:
--
nosy: +giampaolo.rodola
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13585
___
___
Oleg Plakhotnyuk oleg...@gmail.com added the comment:
Forget about patch must be applied before thing.
I've made independent patch. :-)
--
Added file: http://bugs.python.org/file23935/aifc_numbers_fix.patch
___
Python tracker rep...@bugs.python.org
Changes by Oleg Plakhotnyuk oleg...@gmail.com:
Removed file: http://bugs.python.org/file23932/aifc_numbers_fix.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13589
___
Changes by Giampaolo Rodola' g.rod...@gmail.com:
--
nosy: +giampaolo.rodola
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11647
___
___
Raul Morales raul...@gmail.com added the comment:
I use a similar code in my scripts, but I thought it could be useful to have
this feature built into python.
If you prefer subclassing for compression, what about a compressing subclass
built into logging package?
If you think it is a good
Filip Gruszczyński grusz...@gmail.com added the comment:
I have tried to port patch to python2.7, but apparently I must be doing
something wrong, because while most tests pass, several fail (including tests
for unittest itself).
I would like to continue working on this test, but incoming week
Florent Xicluna florent.xicl...@gmail.com added the comment:
Patch is ready for python 3.4
:-)
--
keywords: +patch
nosy: +flox
Added file: http://bugs.python.org/file23936/issue9856_python-3.4.diff
___
Python tracker rep...@bugs.python.org
Sven Marnach s...@marnach.net added the comment:
There is actually a second thread on python-ideas on a very similar topic, see
http://mail.python.org/pipermail/python-ideas/2011-December/013021.html
The two main advantages of the proposed CleanupManager over try/finally blocks
are
1.
Nikolaus Rath nikol...@rath.org added the comment:
On 12/12/2011 03:31 PM, Sven Marnach wrote:
Adding this as a cookbook recipe first seems like a good idea.
This is the second time that this is mentioned, so I would be very
grateful if someone could elucidate what adding as a cookbook recipe
Eric Snow ericsnowcurren...@gmail.com added the comment:
Check out: http://code.activestate.com/recipes/
--
nosy: +eric.snow
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13585
___
Vinay Sajip vinay_sa...@yahoo.co.uk added the comment:
I have worked out a possible approach. I will post about it soon, and add a
link to it from this issue.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13516
New submission from K Richard Pixley r...@noir.com:
Install the Python-2.7.2 mac installer for Lion on Lion.
Then attempt easy_install -U psutil. I get:
za-dc-dev/bin/easy_install -U psutil
install_dir
1 - 100 of 134 matches
Mail list logo