Moritz Lampert
Moritz Lampert
### 🐛 Describe the bug Registering custom configs in GraphGym does not work. It is already not possible to access the custom configs that are specified in the example. To...
Currently the method `MultiOrderModel.lift_order_edge_index(...)` requires `num_nodes` as argument. If you set this incorrectly, you will get an error as follows: ``` IndexError Traceback (most recent call last) Cell In[33], line...
As I understand both methods, `get_snapshot` should return a `TemporalGraph` that contains all edges from the original graph that occur in the time window from `start` to `end`. This means...
@VincenzoPerri raised the issue that we currently cannot add a DAG that should contain the same node at multiple time stamps. We already fixed this issue for `append_walk` by reindexing...
Due to the new internal representation of `node_ids` in `IndexMap` as `numpy.array`, there are sometimes some unintended conversions from integers to booleans happening. I also noticed that the ordering of...
The methods - `lift_order_edge_index` - `lift_order_edge_index_weighted` - `aggregate_edge_index` potentially have uses outside of `MultiOrderModel`, e.g. for the centrality calculations. Since they are static methods anyway, it would make sense to...
Currently files containing a class are named in CamelCase. We should follow [PEP8 convention](https://peps.python.org/pep-0008/#package-and-module-names) and name them in lowercase because this can otherwise lead to confusion during imports. Example: You...
The method `IndexMap.to_idxs(...)` is meant as a shortcut if you want to convert an iterable of ids to the corresponding idx. In some cases one might want to input an...
As we now have almost all of the core functions implemented in `torch` operations that can utilize the GPU, we have fixed most runtime issues and now run into the...
In the previous versions, we saved the number of times each walk or DAG appeared as a count/weight. This would translate to a graph feature for each PyG `Data` object....