[issue9285] Add a profile decorator to profile and cProfile

2017-04-22 Thread Louie Lu
Louie Lu added the comment: Giampaolo, the assertion is still worked good, and no need to remove them. The assertion is to prevent dispatch return too more, to return upper then when the profiler was created. The problem why profile __enter__ can't work, is because it misses the

[issue9285] Add a profile decorator to profile and cProfile

2017-04-22 Thread Giampaolo Rodola'
Giampaolo Rodola' added the comment: Original patch still applies. Not sure if we should continue with that one or with your new PR. The original assertion error is still there. I CCed Tim Peters as it appears he's the one who originally added it in 2001 - maybe he has some clue. --

[issue9285] Add a profile decorator to profile and cProfile

2017-04-20 Thread Louie Lu
Changes by Louie Lu : -- pull_requests: +1335 ___ Python tracker ___ ___ Python-bugs-list

[issue9285] Add a profile decorator to profile and cProfile

2017-02-25 Thread Fred L. Drake, Jr.
Changes by Fred L. Drake, Jr. : -- nosy: -fdrake ___ Python tracker ___ ___

[issue9285] Add a profile decorator to profile and cProfile

2017-02-24 Thread Louie Lu
Changes by Louie Lu : -- pull_requests: +257 ___ Python tracker ___ ___ Python-bugs-list

[issue9285] Add a profile decorator to profile and cProfile

2017-02-24 Thread Louie Lu
Louie Lu added the comment: giampaolo: it seems that contextmanager will somehow make a bad return in `trace_dispatch_return`: $ ./python tests.py # logging.fatal('%r %r' % (frame.f_code.co_name, self.cur[-3])) CRITICAL:root:'runblock' '' ('profile', 0, '') CRITICAL:root:'__enter__' ''

[issue9285] Add a profile decorator to profile and cProfile

2017-02-24 Thread Giampaolo Rodola'
Giampaolo Rodola' added the comment: The original patch is basically blocked by the cryptic assertion error reported above. It's not clear what it means or how to work around it. -- ___ Python tracker

[issue9285] Add a profile decorator to profile and cProfile

2017-02-24 Thread Guido van Rossum
Changes by Guido van Rossum : -- nosy: -gvanrossum ___ Python tracker ___ ___

[issue9285] Add a profile decorator to profile and cProfile

2017-02-24 Thread Louie Lu
Louie Lu added the comment: Ping. Is there any reason why this patch doesn't accept? -- nosy: +louielu ___ Python tracker ___

[issue9285] Add a profile decorator to profile and cProfile

2013-02-25 Thread Éric Araujo
Changes by Éric Araujo mer...@netwok.org: -- nosy: +bjorns ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9285 ___ ___ Python-bugs-list mailing

[issue9285] Add a profile decorator to profile and cProfile

2013-02-25 Thread Éric Araujo
Changes by Éric Araujo mer...@netwok.org: -- versions: +Python 3.4 -Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9285 ___ ___

[issue9285] Add a profile decorator to profile and cProfile

2013-02-25 Thread Giampaolo Rodola'
Giampaolo Rodola' added the comment: Ok, here's an updated patch modeled after: http://hg.python.org/cpython/rev/422169310b7c It works fine with cProfile.py but not with profile.py where I get this exception when I try to use the context manager (tests can be run in order to reproduce it):

[issue9285] Add a profile decorator to profile and cProfile

2013-02-25 Thread Lukas Lueg
Changes by Lukas Lueg lukas.l...@gmail.com: -- nosy: -ebfe ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9285 ___ ___ Python-bugs-list mailing

[issue9285] Add a profile decorator to profile and cProfile

2013-02-12 Thread Giampaolo Rodola'
Changes by Giampaolo Rodola' g.rod...@gmail.com: -- nosy: +georg.brandl ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9285 ___ ___

[issue9285] Add a profile decorator to profile and cProfile

2013-02-12 Thread Giampaolo Rodola'
Giampaolo Rodola' added the comment: A preliminary patch for cProfile.py is in attachment. Will make changes to profile.py later. -- Added file: http://bugs.python.org/file29050/profile.patch ___ Python tracker rep...@bugs.python.org

[issue9285] Add a profile decorator to profile and cProfile

2013-02-11 Thread Guido van Rossum
Guido van Rossum added the comment: Brief comments: - Please don't call it profile -- we already have a module by that name. - Please make it so that both the decorator and context manager can specify a file where to dump the raw data -- basically it needs to have the same functionality as

[issue9285] Add a profile decorator to profile and cProfile

2013-02-11 Thread Giampaolo Rodola'
Giampaolo Rodola' added the comment: Ok, will look into this soon. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9285 ___ ___ Python-bugs-list

[issue9285] Add a profile decorator to profile and cProfile

2012-01-25 Thread Yuval Greenfield
Changes by Yuval Greenfield ubershme...@gmail.com: -- nosy: +ubershmekel ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9285 ___ ___

[issue9285] Add a profile decorator to profile and cProfile

2011-07-13 Thread Éric Araujo
Éric Araujo mer...@netwok.org added the comment: See #8916 for adding standard functionality similar to the decorator module. -- components: +Extension Modules -Benchmarks dependencies: +Move PEP 362 (function signature objects) into inspect nosy: +eric.araujo title: A decorator for

[issue9285] Add a profile decorator to profile and cProfile

2011-07-13 Thread Ezio Melotti
Changes by Ezio Melotti ezio.melo...@gmail.com: -- nosy: +ezio.melotti ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9285 ___ ___ Python-bugs-list

[issue9285] Add a profile decorator to profile and cProfile

2011-07-13 Thread Collin Winter
Changes by Collin Winter coll...@gmail.com: -- nosy: -collinwinter ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9285 ___ ___ Python-bugs-list