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

Investigate nightly CI build issues #1724

Closed
ToucheSir opened this issue Sep 28, 2021 · 5 comments
Closed

Investigate nightly CI build issues #1724

ToucheSir opened this issue Sep 28, 2021 · 5 comments

Comments

@ToucheSir
Copy link
Member

e.g. latest commit of this writing: https://github.com/FluxML/Flux.jl/runs/3708588387

We're currently failing on JuliaArrays/ArrayInterface.jl#205, which should've been fixed in [email protected]. Flux has a compat bound on 3.1 and I was able to add the new version locally without any conflicts, so do we just need to bump the Manifest?

@CarloLucibello
Copy link
Member

Our Manifest is 2 months old, we should get rid of it or automatize the update, the current situation really doesn't make sense.

@CarloLucibello
Copy link
Member

let's see if CI in #1725 goes all green

@DhairyaLGandhi
Copy link
Member

We have updated it in #1708 and can automate updating it too.

@ToucheSir
Copy link
Member Author

Seems like we're back to BoundsError(a=Array{Core.Compiler.VarState, (3,)}[Core.Compiler.VarState(typ=Core.Const(val=ZygoteRules._pullback), undef=false), Core.Compiler.VarState(typ=Zygote.Context, undef=false), Core.Compiler.VarState(typ=Core.Const(val=Main.var"#13"), undef=false)], i=(4,)). Did anyone ever figure out what that was about?

@ToucheSir
Copy link
Member Author

Zygote is good as of FluxML/Zygote.jl#1125, I guess we'll see what's left once a new patch release is cut.

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

No branches or pull requests

3 participants