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

Warnings about unused type variable #598

Closed
mcabbott opened this issue Dec 8, 2022 · 3 comments
Closed

Warnings about unused type variable #598

mcabbott opened this issue Dec 8, 2022 · 3 comments
Labels
help wanted Extra attention is needed

Comments

@mcabbott
Copy link
Member

mcabbott commented Dec 8, 2022

These are annoying:

WARNING: method definition for == at /Users/me/.julia/packages/ChainRulesCore/ctmSK/src/tangent_types/tangent.jl:68 declares type variable T but does not use it.
WARNING: method definition for getindex at /Users/me/.julia/packages/ChainRulesCore/ctmSK/src/tangent_types/tangent.jl:120 declares type variable T but does not use it.
WARNING: method definition for getindex at /Users/me/.julia/packages/ChainRulesCore/ctmSK/src/tangent_types/tangent.jl:120 declares type variable P but does not use it.
WARNING: method definition for canonicalize at /Users/me/.julia/packages/ChainRulesCore/ctmSK/src/tangent_types/tangent.jl:240 declares type variable L but does not use it.
WARNING: method definition for canonicalize at /Users/me/.julia/packages/ChainRulesCore/ctmSK/src/tangent_types/tangent.jl:241 declares type variable L but does not use it.
@mcabbott mcabbott added the help wanted Extra attention is needed label Dec 8, 2022
@devmotion
Copy link
Member

I think that was fixed in #584? Maybe you use an old version of ChainRulesCore? For instance,

WARNING: method definition for == at /Users/me/.julia/packages/ChainRulesCore/ctmSK/src/tangent_types/tangent.jl:68 declares type variable T but does not use it.

refers to a type variable that was fixed in that PR and does not exist anymore on the master branch:

function Base.:(==)(a::Tangent{P}, b::Tangent{P}) where {P}

@oxinabox
Copy link
Member

oxinabox commented Dec 8, 2022

None of those exist on the master branch. So I am guessing must be a old version

@oxinabox oxinabox closed this as completed Dec 8, 2022
@mcabbott
Copy link
Member Author

mcabbott commented Dec 8, 2022

Sorry, I can't reconstruct but agree I must have got an old version somewhere.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants