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

chore/ci: Print if stamp is missing #124

Draft
wants to merge 101 commits into
base: master
Choose a base branch
from

Conversation

I-Al-Istannen
Copy link
Collaborator

This is just a small test because it looks like the CI cache does not contain the relevant stamp file (based on other PRs).

ZuseZ4 and others added 25 commits April 1, 2024 14:59
* fix panic calling code and iteration order

* handle rust name mangling

* fix iteration order
* Warn if the shadow will have internal immutability

* improve error msg
* impl new modes for higher order ad

* fix fwd void ret case
…t checked out (EnzymeAD#107)

* chore: Use correct commit hash as github cache key

* chore: Clone with submodules

This will also initialize all the "doc" submodules, which might prove
too much.
* finish method / trait support
* fix fwd test case

* simplify
* ci: update rustbook tests to use ENYZME_LOOSE_TYPES

* Update .github/workflows/enzyme-ci.yml

Co-authored-by: Jed Brown <jed@jedbrown.org>

---------

Co-authored-by: Manuel Drehwald <git@manuel.drehwald.info>
* Update enzyme-ci.yml

We don't need to build LLD (takes 2.5 min), we just want to use some LLD for linking. We also don't need llvm plugins.

* Update enzyme-ci.yml

* Update .github/workflows/enzyme-ci.yml

Co-authored-by: Jed Brown <jed@jedbrown.org>

* Update enzyme-ci.yml

* Update enzyme-ci.yml

or will it be --enable-use-lld? stay tuned.

* Update enzyme-ci.yml

* Update enzyme-ci.yml

authored-by: @jedbrown

* Update enzyme-ci.yml

* Update enzyme-ci.yml

* Update .github/workflows/enzyme-ci.yml

Co-authored-by: Tim Gymnich <tim@gymni.ch>

* Update enzyme-ci.yml

* Use vendored llvm

* Update enzyme-ci.yml

* Update enzyme-ci.yml

* Update enzyme-ci.yml

* Update enzyme-ci.yml

* Revert "Update enzyme-ci.yml"

This reverts commit 9ec1139.

* Revert "Update enzyme-ci.yml"

This reverts commit 84fe269.

* Fxing lld rebuild
Co-authored-by: I-Al-Istannen <i-al-istannen@users.noreply.github.com>

---------

Co-authored-by: I-Al-Istannen <i-al-istannen@users.noreply.github.com>
Co-authored-by: Jed Brown <jed@jedbrown.org>
Co-authored-by: Tim Gymnich <tim@gymni.ch>
Co-authored-by: William Moses <gh@wsmoses.com>
Co-authored-by: Tim Gymnich <tgymnich@icloud.com>
* Be a bit more aggressive with caching
@I-Al-Istannen
Copy link
Collaborator Author

I-Al-Istannen commented May 8, 2024

This is indeed the case. I am not quite sure how it uploaded a partial cache (or maybe restoring is incomplete?), but GitHub does not seem to allow inspecting caches.
I guess deleting the cache and running it again on the master branch is the way to solve that (e.g. by rerunning the workflow from the UI for some commit on master). Caches are scoped by the branch that created them and only passed down to children.

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.

3 participants