IBM / Multi-GNN

Multi-GNN architectures for Anti-Money Laundering.
Apache License 2.0
57 stars 21 forks source link

Bug of Normalisation #10

Open Linnore opened 4 months ago

Linnore commented 4 months ago

https://github.com/IBM/Multi-GNN/blob/252b0252afca109d1d216c411c59ff70753b25fc/data_loading.py#L108-L127

The data normalisation in line 127 does not normalise the last edge attribute, which corresponds to "payment format" . This is a desired behaviour if no ports nor time deltas are added. However, when ports or time deltas are added, the last edge attribute would not be "payment format" anymore and required normalisation.