-
-
Notifications
You must be signed in to change notification settings - Fork 87
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
Raindrop implementation questions #299
Comments
Hi there 👋, Thank you so much for your attention to PyPOTS! You can follow me on GitHub to receive the latest news of PyPOTS. If you find PyPOTS helpful to your work, please star⭐️ this repository. Your star is your recognition, which can help more people notice PyPOTS and grow PyPOTS community. It matters and is definitely a kind of contribution to the community. I have received your message and will respond ASAP. Thank you for your patience! 😃 Best, |
Hi Lucas, thanks for raising this discussion. We make code logic the same as the official implementation https://github.com/mims-harvard/Raindrop. Yes, some objects and functions don't get used in the model processing procedure. They are kept here in case the official repo gets updated. |
Hey, thank you for your answer! I'll try to reach out to the original authors, then :) Best, |
Lucas has opened an issue in the Raindrop official repo mims-harvard/Raindrop#11. I'm closing this issue for now. If there is any further related problem that should be fixed in PyPOTS, reopening will be welcome. |
Issue description
Hi all, and thanks for such a great package!
I'm currently working on the raindrop architecture, and I have a couple of questions regarding your implementation of the message passing algorithm included here:
The first is where is the message_selfattention method (implementing as far as I can read the attention mechanism described in the paper) used in the code. I tried inspecting it in detail, but I couldn't find any calls to it, or methods with that name in the underlying MessagePassing class from pyg.
The second relates to the L2 regularization parameter on the distances between the graphs. It seems that the computation of the distance is commented out in the file (lines 221, 222). Is the regulariser used and called elsewhere? Or was it removed in later iterations of the model?
Best, and thank you very much once again!
Lucas
The text was updated successfully, but these errors were encountered: