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

Speed up environment setup in CI through local cache/environment #135

Open
jgoeders opened this issue Mar 23, 2023 · 3 comments · Fixed by #250
Open

Speed up environment setup in CI through local cache/environment #135

jgoeders opened this issue Mar 23, 2023 · 3 comments · Fixed by #250

Comments

@jgoeders
Copy link
Member

No description provided.

@jgoeders
Copy link
Member Author

jgoeders commented Jun 1, 2023

It seems to be quite a bit faster now that rapidwright only downloads data files on demand.

@jgoeders
Copy link
Member Author

jgoeders commented Jun 9, 2023

The environment setup action used to take a long time because it downloaded all of the rapidwrite data files. Now the data files are downloaded on demand so it's much faster.

I believe the new longest running setup step is fasm2bels. To make this faster, I think we would want to create a file once the fasm2bels set up completed correctly, and the file would probably contain the hash of the fasm2bels submodule, such that if that file didn't exist or was the wrong version, you would run the setup again, otherwise it could be skipped. (edited)

@jgoeders jgoeders linked a pull request Jul 21, 2023 that will close this issue
@jgoeders
Copy link
Member Author

I've simplified the caching mechanism in #385. Once that is merged we ought to add a similar caching mechanism for yosys. Since much of the code will be duplicated, perhaps we can have a generic caching script for submodules in CI.

@jgoeders jgoeders reopened this Dec 16, 2023
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 a pull request may close this issue.

1 participant