Skip to content
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

Jump to bottom

[fud2,tb] Implement testbench integration #2181

Open
wants to merge 25 commits into
base: main
Choose a base branch
from
Open

[fud2,tb] Implement testbench integration #2181

wants to merge 25 commits into from

Conversation

ethanuppal
Copy link
Contributor

I'm not good at fud2 stuff but I think I'm on the right track

@ethanuppal ethanuppal self-assigned this Jun 30, 2024
@ethanuppal ethanuppal added Status: In progress Issue is being worked on rust labels Jun 30, 2024
@ethanuppal
Copy link
Contributor Author

Ideally I'd like to move the logic in the tb tool into fud2, but I don't really see that being possible.

@ethanuppal ethanuppal requested review from sampsyo and jku20 June 30, 2024 06:11
@ethanuppal
Copy link
Contributor Author

Also closes #2162

@ethanuppal ethanuppal linked an issue Jun 30, 2024 that may be closed by this pull request
Toward a calyx testbench #2162
Open
@ethanuppal
Copy link
Contributor Author

A TODO thing would be to take advantage of the fact that the verilog incoming is calyx and write a new tester that allows you to interface with calyx components preserving the invariants (in addition to just being able to use cocotb and verilator as currently)

@ethanuppal ethanuppal changed the title Implement testbench integration [fud2,tb] Implement testbench integration Jul 3, 2024
@ethanuppal
Copy link
Contributor Author

Waiting on #2193

@rachitnigam rachitnigam removed the rust label Jul 22, 2024
@ethanuppal
Copy link
Contributor Author

fud2 tests failing because of I hack I added before multi-input ops were a thing. @jku20, I assume they are now?

@ethanuppal
Copy link
Contributor Author

For some reason, it seems all my commits became unverified?

@ethanuppal
Copy link
Contributor Author

Now they're verified again...

@rachitnigam
Copy link
Contributor

@ethanuppal what's the status of this PR and what would it take to get it merged?

@ethanuppal
Copy link
Contributor Author

ethanuppal commented Aug 5, 2024 •
edited
Loading

@rachitnigam I've been working with @EclecticGriffin on using cider2 programmatically; once I have the adder component test working (under tools/tb/examples/calyx/), I'll fix all the issues so that it can be merged.

@ethanuppal
Copy link
Contributor Author

@rachitnigam it works!! (for main components due to a limitation in cider right now where you can't run anything else; a hack involves just modifying the Context by setting the entrypoint string

@ethanuppal ethanuppal removed the Status: In progress Issue is being worked on label Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sampsyo sampsyo Awaiting requested review from sampsyo

@jku20 jku20 Awaiting requested review from jku20

@EclecticGriffin EclecticGriffin Awaiting requested review from EclecticGriffin

Assignees

@ethanuppal ethanuppal

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Toward a calyx testbench
2 participants
@ethanuppal @rachitnigam