BNN-UPC / BNN-UPC/MARL-GNN-TE

The unsorted_segment_sum’s parameter of the message_aggregation in the actor.py file seems incorrect

Open
#1 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
17
Forks
8
PR merge metrics
No merged PRs in 30d

Description

On lines 120,122 and 123, tf.math.unsorted_segment_sum(messages, self.outcoming_links, num_segments=self.n_links), 'self.outcoming_links' seems to change to 'self.incoming_links'. Using 'self.incoming_links' seems to conform to the aggregation function described in the paper, which adds up the messages {m (h, h_neighborhood)} of adjacent edges

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.