Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

gh-141999: handle KeyboardInterrupt when sampling and print the result#142000

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

@yihong0618
Copy link
Contributor

@yihong0618yihong0618 commentedNov 27, 2025
edited by bedevere-appbot
Loading

cyliu-phy reacted with thumbs up emoji
Signed-off-by: yihong0618 <zouzou0208@gmail.com>
@pablogsal
Copy link
Member

@yihong0618 Thanks a lot for the PR! I think we are going to need a small test. Let me know if you are having problems or you have questions and I can try to help with it

Signed-off-by: yihong0618 <zouzou0208@gmail.com>
@yihong0618
Copy link
ContributorAuthor

yihong0618 commentedNov 28, 2025
edited
Loading

@yihong0618 Thanks a lot for the PR! I think we are going to need a small test. Let me know if you are having problems or you have questions and I can try to help with it

Thank you and test added

without this patch test

image

with this patch this test

(.venv) ➜  cpython git:(hy/handle_ctrl_c_in_sampling) ./python.exe -m unittest test.test_profiling.test_sampling_profiler.test_profiler...................----------------------------------------------------------------------Ran 19 tests in 0.009sOK

@pablogsalpablogsal added the 🔨 test-with-buildbotsTest PR w/ buildbots; report in status section labelNov 30, 2025
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by@pablogsal for commitf825938 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F142000%2Fmerge

If you want to schedule another build, you need to add the🔨 test-with-buildbots label again.

@bedevere-botbedevere-bot removed the 🔨 test-with-buildbotsTest PR w/ buildbots; report in status section labelNov 30, 2025
@pablogsalpablogsal merged commit056d6c5 intopython:mainNov 30, 2025
54 of 55 checks passed
@pablogsal
Copy link
Member

Thanks a lot for the PR@yihong0618! great work !

yihong0618 reacted with heart emoji

@yihong0618
Copy link
ContributorAuthor

Thanks a lot for the PR@yihong0618! great work !

Thank you!

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@pablogsalpablogsalpablogsal approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@yihong0618@pablogsal@bedevere-bot@gpshead

[8]ページ先頭

©2009-2025 Movatter.jp