search

divelab / DIG

A library for graph deep learning research
https://diveintographs.readthedocs.io/
GNU General Public License v3.0
1.84k stars 281 forks source link

Captum version requirement outdated? #141

Closed jan0b closed 1 year ago

jan0b commented 2 years ago

The version requirement for captum is captum==0.2.0. As version 0.5.0 is already published, is there any specific reason for the strong version requirement in DIG?

Oceanusity commented 2 years ago

Hello, I think it's fine to update the captum. But since some APIs may change across different version, 0.2.0 can make sure that this problems won't happen.

CM-BF commented 1 year ago

The version requirement for captum is captum==0.2.0. As version 0.5.0 is already published, is there any specific reason for the strong version requirement in DIG?

In our code, some low-level functions in capture are used to construct graph-specific explainers. The cost of updating it to a newer version is high since it is not as easy as "changing the high-level API signature". We won't change its version without necessity.

Please let me know if any questions!