torchdyn icon indicating copy to clipboard operation
torchdyn copied to clipboard

Bugs in the neuralde.py for NeuralODE class

Open Charlie-Xiaoqi opened this issue 3 years ago • 3 comments

When looking at the source code under /torchdyn/core/neuralde.py, I found that the NeuralODE class is missing defunc property. And this bug results in an error for /tutorials/module4-model/m4e_lagrangian_nets.ipynb. The notebook can not be run successfully because it requires NeuralODE class to have the defunc property.

Charlie-Xiaoqi avatar Sep 14 '22 18:09 Charlie-Xiaoqi

I have also encountered this problem. Did you manage to solve this in any way?

shaido987 avatar Dec 06 '22 03:12 shaido987

Hey, @Zymrael do you mind providing insight on this issue, I know the neuralde.py API has changed according to the Changelog but I thought it was backward compatible. I think even when I tried to use the torchdyn older version like 0.31, 0.3, 0.4 and etc still the HNN and LNN examples won't run. Any input will be appreciated, Thanks

amdee avatar Dec 16 '22 11:12 amdee

@Charlie-Xiaoqi @shaido987 @amdee one year later, the issue is not solved

oselin avatar Dec 04 '23 09:12 oselin