TopoModelX icon indicating copy to clipboard operation
TopoModelX copied to clipboard

Published 20 hours ago • verified publisher icon pyt-team

Review and rewrite models that are not using topomodelx.base.conv primitives

Open levtelyatnikov opened this issue 8 months ago • 0 comments

Examine the models to determine whether it was necessary to avoid using primitives from topomodelx.base.conv. If it turns out that avoiding them was not justified, update the models by incorporating these topomodelx.base.conv primitives.

  • [ ] UniGCNIILayer
  • [ ] UniGINLayer
  • [ ] UniSAGELayer
  • [ ] HyperGAT (Note: HyperGat precomputes normalization during model initialization. Hence, it is impossible to use for graph-level tasks when the incidence matrices are different.)
  • [ ] HNHN (Note: HNHN takes as input incidence matrix to employ normalization possibly, it can be moved into forward pass to allow different incidence matrices)
  • [ ] HyperSAGE (Note: 1) not train/validation modes as well as node sampling, 2) slow due to loops and .to(device) to allow gpu )

levtelyatnikov avatar Nov 01 '23 15:11 levtelyatnikov