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

use Flux.@layer instead of Flux.@functor #452

Closed

CarloLucibello opened this issue Jul 24, 2024 · 0 comments · Fixed by #484

Milestone

Member

CarloLucibello commented Jul 24, 2024 •

edited

Loading

This would get us fancy printing with model parameters count and suppress the warnings

┌ Warning: This type should probably now use `Flux.@layer` instead of `@functor`: WithGraph{SAGEConv{Matrix{Float32}, Vector{Float32}, typeof(identity), typeof(mean)}, GNNGraph{Tuple{Vector{Int64}, Vector{Int64}, Nothing}}}

The text was updated successfully, but these errors were encountered:

CarloLucibello added this to the v1.0 milestone

This was referenced Aug 7, 2024

@functor ->@layer #484

Merged

@functor default for GNNLayers #288

Closed

CarloLucibello closed this as completed in #484

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment