Skip to content

Ensure IPython's HistoryManager threads don't leak #1361

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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Draft
davidbrochart wants to merge 3 commits into
base: anyio
Choose a base branch
from
Draft

Ensure IPython's HistoryManager threads don't leak #1361

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
6dd2dfb
Ensure IPython's HistoryManager threads don't leak
davidbrochart Feb 26, 2025
10d7abb
Merge branch 'anyio' into history-manager
Carreau May 7, 2026
0ca68e4
Apply suggestion from @Carreau
Carreau May 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions tests/conftest.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
import zmq_anyio
from anyio import create_memory_object_stream, create_task_group, sleep
from anyio.streams.memory import MemoryObjectReceiveStream, MemoryObjectSendStream
from IPython.core.history import HistoryManager
from jupyter_client.session import Session

from ipykernel.ipkernel import IPythonKernel
Expand All @@ -34,6 +35,9 @@ def _garbage_collection(request):
except ModuleNotFoundError:
tracemalloc = None

# ensure we don't leak history managers
HistoryManager._max_inst = 1
Comment thread
Carreau marked this conversation as resolved.
Outdated

pytestmark = pytest.mark.anyio


Expand Down
Loading