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

import errors in gcnlib_cuda and fuseGNN #2

Open
fishmingyu opened this issue Aug 2, 2021 · 5 comments
Open

import errors in gcnlib_cuda and fuseGNN #2

fishmingyu opened this issue Aug 2, 2021 · 5 comments

Comments

@fishmingyu
Copy link

fishmingyu commented Aug 2, 2021

We run install.sh in src file, only to find we couldn't import the fuseGNN module.
In pip list we could find the following module:

fuseGNN                           0.0.1

However, we can not import the fuseGNN in python REPL.

>>> import fuseGNN
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'fuseGNN'

What's more, when we try to run the model with python training_main.py --model GAT --data CiteSeer --mode gas, it will also lead to another import error in gcnlib_cuda.

Traceback (most recent call last):
  File "training_main.py", line 7, in <module>
    from fuseGNN.modules import GCN, gcn_config, GAT, gat_config
  File "/home/yzm18/gcnLib/fuseGNN/modules/__init__.py", line 1, in <module>
    from fuseGNN.modules.gcn import GCN, gcn_config
  File "/home/yzm18/gcnLib/fuseGNN/modules/gcn.py", line 3, in <module>
    from fuseGNN.convs import geoGCNConv, refGCNConv, garGCNConv, gasGCNConv
  File "/home/yzm18/gcnLib/fuseGNN/convs/__init__.py", line 1, in <module>
    from fuseGNN.convs.gcn_conv import geoGCNConv, refGCNConv, garGCNConv, gasGCNConv
  File "/home/yzm18/gcnLib/fuseGNN/convs/gcn_conv.py", line 3, in <module>
    import gcnlib_cuda
ModuleNotFoundError: No module named 'gcnlib_cuda'

We expect that the detailed running configuration could be shown in the README.md to help us reproduce experimental results easily.

@fishmingyu fishmingyu changed the title import error in gcnlib_cuda and fuseGNN import errors in gcnlib_cuda and fuseGNN Aug 2, 2021
@apuaaChen
Copy link
Owner

I fixed the import error, you may try it again.
For the fuseGNN module, I didn't pack the fuseGNN into the pip package. You may try something like

import torch
import fgnn_agg

For the running configuration, the training_main.py along with other source files should be a good example. I will add more instructions when I have some bandwidth.

@fishmingyu
Copy link
Author

Thanks for your update.
But what about gcnlib_cuda.dropout in gcnLib/fuseGNN/dropout.py?

@fishmingyu
Copy link
Author

Thanks for your update.
But what about gcnlib_cuda.dropout in gcnLib/fuseGNN/dropout.py?

I mean this code below is still puzzling.

out, mask = gcnlib_cuda.dropout(input_, rate, training, None)

@apuaaChen
Copy link
Owner

It is only used in the reference module for debugging. So it is not important

@fishmingyu
Copy link
Author

By the way, you seem to forget to change the default data_path in the following code

parser.add_argument('--data_path', type=str, default='/raid/datasets/GNN/', help='the path to datasets')

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