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

Reproduced results of ViG for different graph types is confusing. #185

Open
tdzdog opened this issue Mar 2, 2023 · 1 comment
Open

Reproduced results of ViG for different graph types is confusing. #185

tdzdog opened this issue Mar 2, 2023 · 1 comment

Comments

@tdzdog
Copy link

tdzdog commented Mar 2, 2023

I have investigated the influence of graph types for ViG. However, my experiment results are confusing and different from the data in paper. As shown in Table 6, EdgeConv has highest FLOPS and accuracy. However, according to my results, the results of MaxRelative, GraphSage and EdgeConv are 74.42, 74.46 and 74.24 (ViG-ti). It shows EdgeConv has higher computation but lower accuracy. I also tried PrymaidViG-ti and ViG-s, all results show the high-computation graph (EdgeConv or GraphSage) is worse than the low-computation graph (MaxRelative). This is confusing and I am wondering why. This makes me doubt if the graph architecture necessary enough? Can you release the pretrained models for other graph structures than MaxRelative?

@iamhankai
Copy link
Member

In my experience, MaxRelative is also the best option. If you can propose better alternative than MaxRelative, it would be exciting!

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