Re: [myhdl-list] VHDL Cosim Initial Thoughts

[Daryl W. <dw...@ou...>]

I have a suggestion for a change to the file _Simulation.py 
that will help my socket implementation succeed. Right now, 
the _finalize method of the Simulation class contains two 
lines (76 and 77) that explicitly close the cosim._rt and 
cosim._wf pipes... Also line 78 waits for the child process. 
I think it would be better to call a cosim._finalize method 
that performs these operations on its own member data 
(preserving encapsulation of the Cosimulation object). 

I can do a temporary modification for myself, but I'd like 
to maintain as much compatibility with MyHDL as possible so 
I can benefit from future updates from the MyHDL community.
>From the user viewpoint, the behavior would be identical. 
At some point, I may want to propose a MEP with socket 
support added to Cosimulation (either as a subclass or 
built into the main class) in the main repository if people 
think this implementation will be useful to them, but I 
want a proof-of-concept first... Just to ensure it is 
feasible.

Any thoughts/feedback? Is posting here the right place for 
this type of suggestion? 

Thanks,
Daryl