[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 simulate_call

[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. -- v

[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 mailing list Unsubscribe: https://mail.pytho

[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 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.p

[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 mailing list Unsubscribe: https://mail.python

[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__' '' ('profi

[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 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail

[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 ___ ___ Pytho

[issue9285] Add a profile decorator to profile and cProfile

2013-03-14 Thread Giampaolo Rodola'
Changes by Giampaolo Rodola' : -- nosy: +benjamin.peterson, fdrake, nbastin, rhettinger, tim_one ___ Python tracker ___ ___ Python-bugs

[issue9285] Add a profile decorator to profile and cProfile

2013-02-25 Thread Lukas Lueg
Changes by Lukas Lueg : -- nosy: -ebfe ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/m

[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 Éric Araujo
Changes by Éric Araujo : -- versions: +Python 3.4 -Python 3.3 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: ht

[issue9285] Add a profile decorator to profile and cProfile

2013-02-25 Thread Éric Araujo
Changes by Éric Araujo : -- nosy: +bjorns ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org

[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 ___

[issue9285] Add a profile decorator to profile and cProfile

2013-02-12 Thread Giampaolo Rodola'
Changes by Giampaolo Rodola' : -- nosy: +georg.brandl ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mai

[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 ___ ___ Python-bugs-list mailing list Unsubscr

[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 th

[issue9285] Add a profile decorator to profile and cProfile

2012-01-25 Thread Yuval Greenfield
Changes by Yuval Greenfield : -- nosy: +ubershmekel ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.

[issue9285] Add a profile decorator to profile and cProfile

2011-07-13 Thread Collin Winter
Changes by Collin Winter : -- nosy: -collinwinter ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.p

[issue9285] Add a profile decorator to profile and cProfile

2011-07-13 Thread Ezio Melotti
Changes by Ezio Melotti : -- nosy: +ezio.melotti ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pyt

[issue9285] Add a profile decorator to profile and cProfile

2011-07-13 Thread Éric Araujo
Éric Araujo 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 cProfile and profile