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

Error: tensor size does match and some other question #3

Open
gary-young opened this issue Feb 8, 2022 · 1 comment
Open

Error: tensor size does match and some other question #3

gary-young opened this issue Feb 8, 2022 · 1 comment

Comments

@gary-young
Copy link

gary-young commented Feb 8, 2022

Hi, your work is very interesting and inspiring. However, I had some difficulties when I tried to repeat your experiments.

  1. When I try your first code in run.sh (I just changed the dimension to 100 and keep all other arguments), it raises the error of tensor size mismatching. In fact, it occurs when i in some tasks. (I gauss there are some problems in the CenterIntersection class.)
  2. In your paper, the part of the intersection seems to be non-parameterized, I wonder what the role of the two networks (GaussianOffsetIntersection and CenterIntersection ) on the intersection is in your model.
  3. I gauss self.rank is the 'r' mentioned in Implementation Details in your paper. But why is the dimension of the center also multiplied by the self.rank? (For example, when I set the dimension to 100, the relation_embedding and offset_embedding are both 1600d. 16 is the self.rank.)
  4. It seems that your memory L (size: d * r) instead of S=L*L^T as the inverse of the covariance matrix. However, I find you implement S_1+S_2 by L_1+L_2, but L_1L_1^T + L_2L_2^T is not the same as (L_1+L_2)(L_1+L_2)^T. (line 242-249 in models_gaussian.py)

I am sorry for asking you so many questions. Your work is so wonderful that I really look forward to hearing from you so that I can fully understand your work.

@Akirato
Copy link
Owner

Akirato commented Feb 13, 2022

Hi @gary-young ,

Sorry for the delay in replying. I will run the code with your settings and answer your questions ASAP.
Thanks a lot for studying the work 😄

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