search

DSE-MSU / DeepRobust

A pytorch adversarial library for attack and defense methods on images and graphs
MIT License
994 stars 192 forks source link

Failed to load target_nodes. #44

Closed shaolyy closed 3 years ago

shaolyy commented 3 years ago

https://github.com/DSE-MSU/DeepRobust/blob/f473d0f0bf47c4a091058d32ecee4772481b5752/deeprobust/graph/data/attacked_data.py#L126-L143

Should L127 be before L126? Otherwise, self.target_nodes will always be None.

ChandlerBang commented 3 years ago

Hi shaolyy,

Thank you so much for pointing it out! I've just fixed that issue and pulled a commit https://github.com/DSE-MSU/DeepRobust/commit/d4ebe2d22d2378335f7558a1389dd3e13910fbb2

shaolyy commented 3 years ago

It's my pleasure.