Skip to content

Test ChainRulesCore 1 #166

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

Merged
merged 1 commit into from
Aug 5, 2021
Merged

Test ChainRulesCore 1 #166

merged 1 commit into from
Aug 5, 2021

Conversation

ChrisRackauckas
Copy link
Member

No description provided.

@ChrisRackauckas
Copy link
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
Member

First I'm seeing of this

@ChrisRackauckas
Copy link
Member Author

It's only with ChainRulesCore 1.

@oxinabox
Copy link

oxinabox commented Jul 26, 2021

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