search

cornell-zhang / heterocl

HeteroCL: A Multi-Paradigm Programming Infrastructure for Software-Defined Heterogeneous Computing
https://cornell-zhang.github.io/heterocl/
Apache License 2.0
326 stars 92 forks source link

[CI] CI/CD testing on local servers #424

Closed hecmay closed 2 years ago

hecmay commented 3 years ago

For adding new features

Add feature: add CD/CI testing on our local server. Right now, some test cases requiring EDA tools are skipped in CD/CI testing on CircleCI servers. This PR will add CI/CD testing on our local server, which enables us to verify our design with EDA tools like Vivado HLS or AOC.

How to use the new feature: The local CI/CD testing process is completely automatic.

Detailed description:. For each commit, the webhook on github will automatically inform the local CI/CD runner installed on our local server to pull back the latest changes and run the test cases. The test result will be sent back to github repo and be displayed as the status (i.e., passed or failed) of the commit.

Link to the tests: test_runtime_build.py

hecmay commented 3 years ago

@seanlatias Finally fixed. It turned out the previous seg fault error was caused by a small error in host code generation (i.e., q.finished was missed at the very end).

I believe it is ready for review.

seanlatias commented 2 years ago

Can we just revert tests/test_runtime_build.py? Or is there any reason we need to change this file? Otherwise this PR LGTM.