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

Overall_reweight and Batch_reweight are not running!!! #1

Closed
Mah-Zam opened this issue Nov 5, 2022 · 4 comments
Closed

Overall_reweight and Batch_reweight are not running!!! #1

Mah-Zam opened this issue Nov 5, 2022 · 4 comments

Comments

@Mah-Zam
Copy link

Mah-Zam commented Nov 5, 2022

Hello Dear,
Thank you for sharing your research paper source code here. I think code is only running with "reweight" instead. Otherwise we will get below error:

G2GNN/learn.py", line 192, in train
loss.backward()
UnboundLocalError: local variable 'loss' referenced before assignment

Please correct me if I am wrong.

Thank you so much.

@yuwvandy
Copy link
Owner

yuwvandy commented Nov 5, 2022

Hi Mah,
Thank you for your interesting in our work and code. We have updated our code heavily recently and made changes as follows:
(1) provide the implementation of both reweight and batch-reweight
(2) wrap the search of the nearest neighboring graphs within the dataprocess module to allow parallel sampling
(3) Instead of using the original scatter that is non-deterministic to pool the node representation to the graph representation, we use Segment CSR to implement, allowing the deterministic results.

Please download the most recent codes and run the bash_{dataset}.sh files. And let me know if you have any further issue from there!

@Mah-Zam
Copy link
Author

Mah-Zam commented Nov 5, 2022

Sure. May I know how can I contact you? I have multiple advanced research projects and would like to write some papers. Of course, if you are interested. The results are available. About the Grakel package which you have used in your project, I need to change their libraries to use our dataset while I have changed and generated my script using torch_geometric for new datasets. Do you know how we can do that? Thanks again

@yuwvandy
Copy link
Owner

yuwvandy commented Nov 6, 2022

Sure. May I know how can I contact you? I have multiple advanced research projects and would like to write some papers. Of course, if you are interested. The results are available. About the Grakel package which you have used in your project, I need to change their libraries to use our dataset while I have changed and generated my script using torch_geometric for new datasets. Do you know how we can do that? Thanks again

It is interesting that you are trying to use some of our techniques developed here. My email is yu.wang.1@vanderbilt.edu. Feel free to drop me an email and if you have Wechat, you can also send me your Wechat via email so that we can chat there if it is more convenient for you.

I think this post would provide some insights on how to calculate kernel similarity for customized datasets. ysig/GraKeL#59. However, if the number of graphs/ the size of every single graph is too large, we might need some advanced techniques to deal with the complexity there. We can discuss this furthermore via email.

@Mah-Zam
Copy link
Author

Mah-Zam commented Dec 12, 2022

Hello Yu, I would like to meet you in Wechat. I will wait your response, I sent you an email. Thanks

@yuwvandy yuwvandy closed this as completed Sep 1, 2023
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