Skip to content

Test ChainRulesCore 1 - #166

Merged
YingboMa merged 1 commit into
masterfrom
ChrisRackauckas-patch-1
Aug 5, 2021
Merged

YingboMa merged 1 commit into
masterfrom
ChrisRackauckas-patch-1

Conversation

@ChrisRackauckas

Copy link
Copy Markdown
Member

No description provided.

@ChrisRackauckas

Copy link
Copy Markdown
Member Author

Anyone know what's going on with the chainrulescore update and why it would cause this? @oxinabox @DhairyaLGandhi

https://github.com/SciML/RecursiveArrayTools.jl/pull/166/checks?check_run_id=3161312554#step:6:238

@DhairyaLGandhi

Copy link
Copy Markdown
Member

First I'm seeing of this

@ChrisRackauckas

Copy link
Copy Markdown
Member Author

It's only with ChainRulesCore 1.

@oxinabox

oxinabox commented Jul 26, 2021 •

Copy link
Copy Markdown

CI there is using Zygote v0.4.20
https://github.com/SciML/RecursiveArrayTools.jl/pull/166/checks?check_run_id=3161312554#step:6:103

Which is truely ancient, given the current release is 0.6.17.
I recommend waiting before updating til we have FluxML/Zygote.jl#1035 in
so that a modern version of Zygote works with ChainRules v1.
(I am looking into the remaining errors in the Zygote PR today and tomorrow)

@YingboMa YingboMa closed this Aug 5, 2021
@YingboMa YingboMa reopened this Aug 5, 2021
@YingboMa
YingboMa merged commit be87bf7 into master Aug 5, 2021
@YingboMa
YingboMa deleted the ChrisRackauckas-patch-1 branch August 5, 2021 19:40
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.

4 participants