Issues related to graph injection attacks on NIPA about deeprobust HOT 5 OPEN

Hi,

The input adjacency matrix already contains the injected nodes. Please see the file below.

from deeprobust.

Thank you very much for your answer!
The input adjacency matrix does indeed include injection nodes, but at this point, the degree of the injected nodes is 0 because the injection nodes have not yet been connected to the nodes in the original graph (the "injecting_nodes" function in the "deeprobust/examples/graph/test_nipa. py" file seems to have only changed the shape of the initial adjacency matrix). However, the variable "N" in the "deeprobust/graph/rl/nipa_env. py" file represents the number of nodes with a degree greater than 0, which is the number of nodes in the original graph (the degree of all nodes in the graph I chose to attack is greater than 0), and the variable "self. all_nodes" should represent the index of all nodes in the original graph. I also debugged the code, which confirms my point.

In the above figure, I printed the shapes of the feature matrices before and after injecting nodes. Before injecting nodes, the number of nodes was 2708, and after injecting nodes, the number of nodes was 2735. Therefore, the index of the injected nodes should be 2708-2734. However, I checked the index stored in the variable "self.injected_nodes" in the "deeprobust/graph/rl/nipa_env. py" file, and it looks like this:

The value of the variable "self.injected_nodes" is actually stored between 2681 and 2707.
I hope you can check it out and correct me if I'm wrong.

from deeprobust.

Hi,

The input adjacency matrix already contains the injected nodes. Please see the file below.

DeepRobust/examples/graph/test_nipa.py

Lines 60 to 62 in fb047ac

	def init_setup():
	data = Dataset(root='/tmp/', name=args.dataset, setting='nettack')
	injecting_nodes(data)

Hi,

The input adjacency matrix already contains the injected nodes. Please see the file below.

DeepRobust/examples/graph/test_nipa.py

Lines 60 to 62 in fb047ac

	def init_setup():
	data = Dataset(root='/tmp/', name=args.dataset, setting='nettack')
	injecting_nodes(data)

Hello, could you please reply to me?

from deeprobust.

Hi mmdhhhh,

Thank you for pointing out. I agree that there is a bug in this file. Can you help me fix this issue and submit a pull request to this repo? I appreciate your help.

from deeprobust.

Thank you for your answer！
I will submit a pull request soon.

from deeprobust.