Skip to content

correctness temporalgraph implementationΒ #548

@CarloLucibello

Description

@CarloLucibello

I think our T-GCN implementation is not correct, since we have only a single graphconv inside instead of two.

A reference pytorch implementation is here
https://github.com/martinwhl/T-GCN-PyTorch/blob/main/models/tgcn.py
or here
https://pytorch-geometric-temporal.readthedocs.io/en/latest/_modules/torch_geometric_temporal/nn/recurrent/temporalgcn.html#TGCN

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions