Marlon James
Marlon James
@imphil I think that is essentially what we have now. The more I think about it, the more I think our current behavior is fine as the default. If a...
> `TaskGroup.start_soon`'s `True` would be the best This would mean the default in a `TaskGroup` is to **not** cancel the remaining tasks when a task raises an `Exception`. That's the...
Something like `trio.Nursery` could be used to deal with cleaning up coroutines and triggers. ```python3 async def run_all_tests(): for test in tests: async with cocotb.nursery_thing(): await test() ``` Where any...
If we were to use pytest as the regression manager, we could provide a general command line option that accepts some syntax for parameterizing tests, and apply them as appropriate....
Icarus is returning `vpiNamedBegin` rather than `vpiAlways` (which is supported): ``` 5000.00ns DEBUG cocotb.gpi VpiCbHdl.cpp:893 in next_handle vpi_scan found 'dff.this_name.some_name' 5000.00ns DEBUG cocotb.gpi VpiImpl.cpp:257 in create_gpi_obj_from_handle VPI: Not able to...
@ktbarrett > This is pretty weird. It is finding `this_name` modules within the other_diff module. This points to an issue in Icarus. Looks like for some reason icarus is adding...
I looked into this, and it's completely cocotb's fault: https://github.com/cocotb/cocotb/blob/dc6ad51ebb387d36ed78389d7971717ab9935ec7/cocotb/share/lib/vpi/VpiImpl.cpp#L237-L248 We rely on the **gen scope array** name (**gen scope** prefix) not matching the parent name. We essentially throw away...
I have an old branch where I started going through the tests. I'll give it a look soon.
Aldec doesn't support `cbAtEndOfSimTime`. It should also be noted that some simulators may interpret the _cb_data_p->time_ field differently. Icarus interprets it as an absolute time for `cbAtStartOfSimTime` and `cbAtEndOfSimTime`, but...
Have you tried running the simulation with valgrind to find the source of the memory leak?