Skip to content

At startup, count how many frames Context.run adds to tracebacks#634

Merged
belm0 merged 2 commits intopython-trio:masterfrom
njsmith:count-context-run-frames
Sep 1, 2018
Merged

At startup, count how many frames Context.run adds to tracebacks#634
belm0 merged 2 commits intopython-trio:masterfrom
njsmith:count-context-run-frames

Conversation

@njsmith
Copy link
Copy Markdown
Member

@njsmith njsmith commented Aug 28, 2018

This is kind of ridiculous overkill, but I was thinking about how you
could do it, then wrote some code to experiment, then realized that
I had implemented it so we might as well use it...

@njsmith njsmith changed the title At startup, how many frames Context.run adds to tracebacks At startup, count how many frames Context.run adds to tracebacks Aug 28, 2018
This is kind of ridiculous overkill, but I was thinking about how you
could do it, then wrote some code to experiment, then realized that
I had implemented it so we might as well use it...

See also: python-trio#631
@njsmith njsmith force-pushed the count-context-run-frames branch from e6235cf to b31da5e Compare August 28, 2018 05:58
@njsmith njsmith requested a review from belm0 August 28, 2018 05:58
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 28, 2018

Codecov Report

Merging #634 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #634      +/-   ##
==========================================
+ Coverage   99.31%   99.31%   +<.01%     
==========================================
  Files          93       93              
  Lines       10954    10968      +14     
  Branches      782      783       +1     
==========================================
+ Hits        10879    10893      +14     
  Misses         56       56              
  Partials       19       19
Impacted Files Coverage Δ
trio/_core/_run.py 99.69% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fbb4543...1cfadda. Read the comment docs.

Comment thread trio/_core/_run.py
def function_with_unique_name_xyzzy():
1 / 0

ctx = copy_context()
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand this part

Comment thread trio/_core/_run.py
@belm0
Copy link
Copy Markdown
Member

belm0 commented Aug 31, 2018

@njsmith please go ahead and merge

@belm0 belm0 force-pushed the count-context-run-frames branch from 2b8e081 to 1cfadda Compare September 1, 2018 06:49
@belm0 belm0 merged commit a27696b into python-trio:master Sep 1, 2018
@njsmith njsmith deleted the count-context-run-frames branch September 3, 2018 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants