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.

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

tests: Improve GitHub CI action #2144

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

FlyGoat
Copy link
Contributor

@FlyGoat FlyGoat commented Dec 18, 2024

Improve up GitHub action by bumping software versions, use cache to preserve tool builds, and use Clang to speed up verilator.

@enjoy-digital
Copy link
Owner

Thanks a lot @FlyGoat. Would you mind having the ci.yml changes in another PR? I'm trying to synchronize changes for this between teh different repositories, so will maybe have more delay to look at it.

@FlyGoat FlyGoat changed the title tests: Improve action and include more bus option tests tests: Improve GitHub CI action Dec 20, 2024
@FlyGoat
Copy link
Contributor Author

FlyGoat commented Dec 20, 2024

@enjoy-digital Done, would you be interested to test multiple Python versions with CI? I can implement it as well if you are planning to sync the change between repos.

Thanks

@FlyGoat
Copy link
Contributor Author

FlyGoat commented Jan 10, 2025

Hi @enjoy-digital,

Do you mind help me to merge those PRs so I can wire up those cores in CI?

litex-hub/pythondata-cpu-picorv32#1
litex-hub/pythondata-cpu-mor1kx#2
litex-hub/pythondata-cpu-cv32e40p#3

Or maybe add me to litex-hub so I can help with maintaining pythondata packages :-)

@enjoy-digital
Copy link
Owner

Hi @FlyGoat,

sorry, I just sent you an invite to be able to merge things directly. Feel free to merge these kind of PR directly. In case of doubt, feel free to ask, I'll try to be more reactive :)

@FlyGoat
Copy link
Contributor Author

FlyGoat commented Jan 10, 2025

@enjoy-digital Thanks a lot! I'll get those sorted.

Exposed with new tests.

Signed-off-by: Jiaxun Yang <[email protected]>
Test Python 3.8 and 3.12 as our lower and upper bound of
supported python versions, since testing full python range
would require too much resources.

Signed-off-by: Jiaxun Yang <[email protected]>
Synthesis tested cpu list from litex.soc.cores.cpu to ensure
all CPUs are covered.

Refresh untested_cpu list to reflect latest status.

Signed-off-by: Jiaxun Yang <[email protected]>
@FlyGoat
Copy link
Contributor Author

FlyGoat commented Jan 12, 2025

Hi @enjoy-digital,

This is ready for review. I'll look into individual failures after getting this merged.

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants