-
Notifications
You must be signed in to change notification settings - Fork 17
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
Add cocotb testbench runner #222
base: master
Are you sure you want to change the base?
Conversation
7c86ed5
to
472d3ef
Compare
…naries from the INI timing file before running the simulation. Cleaned code to comply with PEP 8.
Fixing bug where .vcd file was deleted. Bugfix
Using make rather than calling cocotb script directly in CI
run cocotb tests on iris runners
05986f2
to
63b5659
Compare
Note to self: codecov step still shows as passing even when it fails to upload
Currently using the container from this PR PandABlocks/PandABlocks-rootfs#67, may need to change the python version or something. The coverage file generated uses absolute paths which only work inside the container (prefixed by /repos/PandABlocks-FPGA) so using a sed command to make these relative so we can hopefully get line by line breakdowns of coverage. edit: apparently when this was working the codecov-action was running on ubuntu-latest instead of inside the container, and the cocotb tests were done inside a docker run command... |
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
Rebasing Jacob's work on the cocotb testbench runner on master and fixing up the git history a bit (WIP).
I've added a few commits on top to use the pytest runner and upload "cobertura" style coverage from nvc to codecov.
Opening this PR as a draft while I work on fixing up the CI etc