search

epfl-lts2 / pygsp

Graph Signal Processing in Python
https://pygsp.rtfd.io
BSD 3-Clause "New" or "Revised" License
483 stars 93 forks source link

need to add nngraphs to requirements list #10

Closed Gibbsdavidl closed 7 years ago

Gibbsdavidl commented 7 years ago

Hello,

In update_graph_attr(self, *args, **kwargs) you import "from nngraphs import NNGraph" but do not list this in the requirements.txt and the function fails without it.

I believe this comes from torch?

Thanks! -dave

mdeff commented 7 years ago

Hello, nngraphs is the sub-package of pygsp/graphs where the NNGraph class is defined. Are you actually encountering an error?

mdeff commented 7 years ago

There was indeed an issue. ce861c67008efffe3f5dee784d8854d43d5c57e1 fixed it. Thanks for reporting!