-
Notifications
You must be signed in to change notification settings - Fork 40
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
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.
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
cocotb coherence #63
Comments
We need a maintainer willing to work on this repo. And I don't think that cocotb-bus and cocotb should be able to be installed together right now, the setup.py in this repo lists EDIT: Ah, 2.0.dev0 is less than 2.0. |
@ktbarrett @toddstrader , |
I agree @jahagirdar - I think providing a basic set of tools to implement VIPs is a good direction, happy to put in some work to get this functionality alongside the cocotb project. |
@eric-wieser @cmarqu @imphil , |
Good idea, I just enabled them. |
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
Is there an expectation for HEAD-to-HEAD correctness between cocotb and cocotb-bus? Is coherence expected to be best effort? Will the packages be released in tandem so at least releases are known to work together? Also, I'm not sure if this is possible with GitHub CI, but should this repo know when it is broken wrt cocotb's HEAD?
Also also, what's the prognosis for this repo? I see it hasn't had a commit in over a year. Dare I ask, is cocotb-bus dead?
I recently posted #62 when
fork
was removed from cocotb. And now I'm seeing that_AssignmentResult
has been removed with is breakingbus.py
. I haven't dug further since the last round of pruning in cocotb, so I'm not sure what else is busted. I can keep fixing things up, but just want to make sure I'm headed in a meaningful direction.@ktbarrett thoughts?
The text was updated successfully, but these errors were encountered: