Emanuele Rossi
                                            Emanuele Rossi
                                        
                                    Hi @antoniofilipovic, You are correct. In practice, it does not make much of a difference since we're mostly using only 1 graph layer with TGN, but this is indeed incorrect...
Hi @benelot, the reason for the `+1` is that the node indexes start from 0, so if the max index is `max_idx`, it means that in total we have `max_idx...
The node feature are actually added to the memory every time the embedding is computed, see [here](https://github.com/twitter-research/tgn/blob/286815a4e15de6a6d4e0b8e660189e9c3c9f1dbf/modules/embedding_module.py#L98)