Skip to content

sim: call .aclose() on TickTrigger in .until() and .repeat() [0.5 backport]#1591

Merged
whitequark merged 1 commit intoamaranth-lang:v0.5.xfrom
whitequark:sim-aclose-0.5
May 11, 2025
Merged

sim: call .aclose() on TickTrigger in .until() and .repeat() [0.5 backport]#1591
whitequark merged 1 commit intoamaranth-lang:v0.5.xfrom
whitequark:sim-aclose-0.5

Conversation

@whitequark
Copy link
Member

Otherwise, when used together with asyncio, the following warning will be printed for each use of .until()/.repeat():

E: asyncio: Task was destroyed but it is pending!
task: <Task pending name='Task-2' coro=<<async_generator_athrow without __name__>()>>
/usr/lib/python3.13/asyncio/base_events.py:744: RuntimeWarning: coroutine method 'aclose' of 'TickTrigger.__aiter__' was never awaited

Otherwise, when used together with asyncio, the following warning will
be printed for each use of `.until()`/`.repeat()`:

    E: asyncio: Task was destroyed but it is pending!
    task: <Task pending name='Task-2' coro=<<async_generator_athrow without __name__>()>>
    /usr/lib/python3.13/asyncio/base_events.py:744: RuntimeWarning: coroutine method 'aclose' of 'TickTrigger.__aiter__' was never awaited
@whitequark whitequark merged commit a96cd85 into amaranth-lang:v0.5.x May 11, 2025
20 checks passed
@whitequark whitequark deleted the sim-aclose-0.5 branch May 11, 2025 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant