search

lowRISC / ot-sca

Side-channel analysis setup for OpenTitan
Apache License 2.0
28 stars 26 forks source link

[ci] Add Python caching #343

Closed jwnrt closed 7 months ago

jwnrt commented 7 months ago

This did not seem to provide any benefit. Possibly because many of our dependencies are from git and don't get loaded from the cache? I'm not sure. Closing.

vogelpi commented 7 months ago

@jwnrt can you maybe explain the problem you're trying to solve with this please? Pascal or I might then be able to help. Is this to speed up CI? We're loading some Python dependencies as tarballs from GitHub because they're big. In case we need to speed this up, we could include this in the container image and periodically rebuild the container image.

jwnrt commented 7 months ago

Oh apologies, I'm actually using this pull request to test the huskies in CI work. I don't have permissions to manually run the pipeline, so I'm just force pushing to this PR to trigger it again.

I originally opened it to see if the dependency download time could be reduced, but it's only a minute long so not a priority to optimize right now.

I didn't realise you guys were getting notifications from this, sorry

vogelpi commented 7 months ago

Ah, I see. Thanks for explaining @jwnrt . All good then!