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

issue with DAE #2

Closed
jingyonglin opened this issue Jan 22, 2022 · 1 comment
Closed

issue with DAE #2

jingyonglin opened this issue Jan 22, 2022 · 1 comment

Comments

@jingyonglin
Copy link

compare the generated features with original features, the size must match.
so at model.py
line 30:
self.layers.append(GCNConv_dense(hidden_dim, nclasses))

think it should be
self.layers.append(GCNConv_dense(hidden_dim, in_dim))

@baharefatemi
Copy link
Collaborator

The size match as when creating an instance of a GCN_DAE in lines 229 and 162 in main.py, both in_dim and n_classes are set to nfeats. The reason why not use only one parameter for both is to keep the model general.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants