-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DEQ layer #5
Comments
Food for thought: may need to be careful and do some preprocessing/rounding on weight matrices here, because sometimes graph edge weights that should be identical aren't quite...take the methane molecule for instance:
And obviously a SteadyStateSolution would be a lot more sensitive to this than the result of just a couple discrete convolutions... |
@ChrisRackauckas @YingboMa any updates on whatever's needed under the hood (something with |
I don't think it's the implementation I think it's the method. A different rootfinding method would be needed her for a faster forward pass. The Newton iterations are just unstable so it needs to somehow be stabilized. |
Are you saying a new method needs to be implemented or just that I should try other methods? Also, IIRC even when the forward pass worked, the backwards still didn't. |
Want to get this capability working (reference) eventually.
The text was updated successfully, but these errors were encountered: