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

About the number of network loops #22

Open
yiyihao2000 opened this issue Apr 11, 2024 · 3 comments
Open

About the number of network loops #22

yiyihao2000 opened this issue Apr 11, 2024 · 3 comments

Comments

@yiyihao2000
Copy link

Your work is excellent, I would like to ask a question, may I ask you, I see that the number of layers of the network in the paper is 12, why is the n_line_descriptive_layers=1 in the code, is the part of the network circulating 12 times elsewhere?

@yosungho
Copy link
Owner

Hello, it turned out that I used n_line_descriptive_layers = 1 for the paper result. I had a chance to train the network with a greater number of layers and it helped to enhance matching performance.

@yiyihao2000
Copy link
Author

您好,原来我用 n_line_descriptive_layers = 1 作为论文结果。我有机会用更多的层来训练网络,这有助于提高匹配性能。

You mentioned in your paper that the lineTransformer network has 12 layers, where is this reflected in the code, maybe my negligence caused me not to find it, please give me a hint, thank you very much

@yosungho
Copy link
Owner

yosungho commented Apr 14, 2024

I used only a single 1 layer even when I got the paper result. The paper should be corrected to be 1 layer.

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