Skip to content

Conversation

@yihong0618
Copy link
Contributor

@yihong0618 yihong0618 commented Nov 27, 2025

before this patch
image

after this patch
image

Signed-off-by: yihong0618 <zouzou0208@gmail.com>
@gpshead gpshead requested a review from pablogsal November 27, 2025 18:34
@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
Contributor Author

yihong0618 commented Nov 28, 2025

@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.009s

OK

@pablogsal pablogsal added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 30, 2025
@bedevere-bot
Copy link

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

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-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 30, 2025
@pablogsal pablogsal merged commit 056d6c5 into python:main Nov 30, 2025
54 of 55 checks passed
@pablogsal
Copy link
Member

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

@yihong0618
Copy link
Contributor Author

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

Thank you!

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.

4 participants