-
Notifications
You must be signed in to change notification settings - Fork 0
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
PINNx vs DeepXDE #19
Comments
Dear Lu Lu, thanks you so much for your message. I'm sorry for not prominently thanking DeepXDE, but in fact, we clearly stated in About PINNx that pinxx We are also highly interesting in merging I will pull a PR and try to integrate pinxx into deepxde in these days. I am looking forward to your response to this upcoming PR. Thank you once again. |
Dear Lu Lu, please see the PR lululxvi/deepxde#1932 for details, in which I drafted a version of merging. |
Sounds great. |
Thank you for sharing the library. I find that PINNx uses a lot of code from DeepXDE. I am OK with this, but it should be clearly acknowledged in the source code and README, in addition to "inspired from [DeepXDE](https://github.com/lululxvi/deepxde".
Also, you and PINNx team are very welcome to contribute code back to DeepXDE. I am happy to merge the new code/design from PINNx to DeepXDE, and maintain the code together.
The text was updated successfully, but these errors were encountered: