Clarify duplicate implementation of SCN/SCN2

[pyt-team]

Clarify duplicate implementation of SCN/SCN2

[ninamiolane]

Thank you very much for stepping in! This is incredibly helpful. I see the two models in Yang et al. 2022 and the confusion arising from their misnaming in Figure 11. Thus:

• SCN: Simplex Convolutional Network (rightmost Yang22c in figure 11): happens to be implemented up to rank 2 in topomodelx.
• SCCN: Simplicial Complex Convolutional Network (leftmost Yang22c in figure 11)

Thus, TODOs for us:

• Rename the leftmost SCN of Figure 11 of the Architectures of Topological Deep Learning paper into SCCN.
• Rename the files of scn_layer -> sccn_layer.py, test_scn_layer.py -> test_sccn_layer.py, scn_train.ipynb -> sccn_train.ipynb

[georg-bn]

Hi, I'm one of the implementers of SCCN.

I believe that SCN and SCCN are two different models from Yang et al 2022. There is some confusion in that they are both labelled SCN in Figure 11 of the Architectures of Topological Deep Learning paper. SCCN should be the leftmost "SCN" in that figure.

[ninamiolane]

We will update the Figure 11 offline.