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.

Sign up for GitHub

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

Jump to bottom

chore/ci: Print if stamp is missing #124

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

chore/ci: Print if stamp is missing #124

wants to merge 101 commits into 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
wip
79d1834
@ZuseZ4
got autodiff macro to do something
12292ca
@ZuseZ4
cleanup, first helper
175d236
@ZuseZ4
adding Enzyme submodule
095eabd
@ZuseZ4
adding extra error message
c3b73ce
@ZuseZ4
adding ffi, not building enzyme yet
e4e3430
@ZuseZ4
Build and link Enzyme durin bootstrap
51e0b65
@ZuseZ4
update rustc_middle, rustc_monomorphize, and (partly) cg_ssa
f253e67
@ZuseZ4
update the backend, cg_ssa and cg_llvm
b394066
@ZuseZ4
add minimal ast based macro parser
6700542
@ZuseZ4
yeet
bf72f16
@ZuseZ4
compiles and runs binaries, but without Enzyme yet
001bc8a
@ZuseZ4
I will clean this up, I promise!
e3ba2d2
@ZuseZ4
remove leftovers
3e89575
@ZuseZ4
mark autodiff_into as builtin and permanently unstable
f17e81d
@ZuseZ4
follow rustc_ naming convention
c97f625
@ZuseZ4
It works (somewhat)
494b102
@ZuseZ4
Fwd: Rename Duplicated -> Dual
66f86a6
@ZuseZ4
properly allow interior instability
59bc30f
@ZuseZ4
make d_fnc decl a bit more precise
12b180e
@ZuseZ4
running x.py fmt
47d6d3c
@ZuseZ4
various fn_decl fixes
e4b7ef1
@ZuseZ4
More precise Enzyme settings
9cbdf68
@ZuseZ4
cleanup, reduce diff
bdc8e5d
@ZuseZ4
cleanups
c8c4ea3
@ZuseZ4
cleanups2
7b0d0f1
@ZuseZ4
make Enzyme build optional
a06caa7
@ZuseZ4
Implement fallback to optionally not link and build Enzyme
2de98a2
@ZuseZ4
adding CI and updating README
293ab97
@ZuseZ4
fix mem leak, fix logic bug, cleanup
4889ad3
ZuseZ4 and others added 22 commits April 1, 2024 21:16
@ZuseZ4
fix panic calling code and iteration order (EnzymeAD#96)
7468cf8 
* fix panic calling code and iteration order

* handle rust name mangling

* fix iteration order
@ZuseZ4
enable runtime activity (EnzymeAD#98)
26c676b
@ZuseZ4
printing (EnzymeAD#99)
683aa31
@ZuseZ4
new flag (EnzymeAD#101)
8e61557
@ZuseZ4
updatenzyme (EnzymeAD#102)
eab555d
@ZuseZ4
Warn if the shadow will have internal immutability (EnzymeAD#103)
1218cb2 
* Warn if the shadow will have internal immutability

* improve error msg
@ZuseZ4
impl new modes for higher order ad (EnzymeAD#106)
1e32009 
* impl new modes for higher order ad

* fix fwd void ret case
@ZuseZ4
adding two more flags (EnzymeAD#109)
b5a3130
@I-Al-Istannen
chore: Use correct commit hash as github cache key if submodule is no…
af4d766 
…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.
@ZuseZ4
support self ty (EnzymeAD#110)
948848d 
* finish method / trait support
@ZuseZ4
add rust error for activity to arg count missmatch (EnzymeAD#111)
ffdbffb
@ZuseZ4
unbreak ActiveOnly return (EnzymeAD#112)
17c772f
@ZuseZ4
handle DualOnly ret more reliably (EnzymeAD#113)
9a411dc
@ZuseZ4
last commit broke the right order of updating attributes (EnzymeAD#114)
db2a069
@ZuseZ4
fix higher-order { float } ret case (EnzymeAD#115)
5747648
@ZuseZ4
fix fwd test case (EnzymeAD#116)
15bfd28 
* fix fwd test case

* simplify
@jedbrown @ZuseZ4
ci: update rustbook tests to use ENYZME_LOOSE_TYPES (EnzymeAD#117)
b445b6c 
* ci: update rustbook tests to use ENYZME_LOOSE_TYPES

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

Co-authored-by: Jed Brown <[email protected]>

---------

Co-authored-by: Manuel Drehwald <[email protected]>
@ZuseZ4 @I-Al-Istannen
@jedbrown @tgymnich @wsmoses
Update enzyme-ci.yml (EnzymeAD#84)
4cae481 
* 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 <[email protected]>

* 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 <[email protected]>

* 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 <[email protected]>

---------

Co-authored-by: I-Al-Istannen <[email protected]>
Co-authored-by: Jed Brown <[email protected]>
Co-authored-by: Tim Gymnich <[email protected]>
Co-authored-by: William Moses <[email protected]>
Co-authored-by: Tim Gymnich <[email protected]>
@I-Al-Istannen
chore: Cache random stuff (EnzymeAD#118)
c9b28e7 
* Be a bit more aggressive with caching
@ZuseZ4
fix ci (EnzymeAD#119)
c00250e
@ZuseZ4
Update README.md (EnzymeAD#120)
75b31f9
@I-Al-Istannen
ci: Print if stamp is missing
fc4ab22
@I-Al-Istannen
Copy link
Collaborator Author

I-Al-Istannen commented May 8, 2024 •
edited
Loading

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.

@ZuseZ4 ZuseZ4 force-pushed the master branch 3 times, most recently from 4f0389e to 012062a Compare October 28, 2024 02:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@I-Al-Istannen @ZuseZ4 @jedbrown