HennyJie / BrainGB

Officially Accepted to IEEE Transactions on Medical Imaging (TMI, IF: 11.037) - Special Issue on Geometric Deep Learning in Medical Imaging.
MIT License
165 stars 42 forks source link

A bug in the brain_dataset.py #15

Closed Gemaxis closed 9 months ago

Gemaxis commented 9 months ago

Hi,

This is excellent work, but I found a bug in brain_dataset.py. At the function named dense_to_ind_val, torch.isnan(adj)==0 will make every existing value turn into True, causing the index fully True which means every brain network is fully connected.

yangji9181 commented 9 months ago

Thanks For the comments. The brain networks are indeed fully connected, which are different from most graphs in other domains.