-
Notifications
You must be signed in to change notification settings - Fork 332
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
Update Makefile to pip caravel-cocotb #307
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add the installation and running inside venv?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like you updated the target names, can you update it in the documentation as well? (make sure that you're updating the correct documentation). Also the name of the test from counter_tests/counter_tests.yaml
to user_proj_tests/user_proj_tests.yaml
, so you need to update the name of the files.
I get this error
I attached the compilation log, it has errors in the include. |
I'm aware of this error. |
remove display info from make file Co-authored-by: Marwan Abbas <67271180+marwaneltoukhy@users.noreply.github.com>
Rename cocotb virtual env Co-authored-by: Marwan Abbas <67271180+marwaneltoukhy@users.noreply.github.com>
new version of caravel-cocotb script. It should fix #303.
Please delete the branch after merge.