-
Notifications
You must be signed in to change notification settings - Fork 74
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
Condition takes only the following keyword BUG #90
Comments
Dear @LoveFrootLoops, thanks for reporting! |
@ndem0 I am on it to change the tutorial |
👋🏻 @LoveFrootLoops Thank you for the report. Recently we changed the class Now the tutorials should all work, hope it helps. |
@ndem0 If for you it is ok, it can be merged |
Thanks for the fast response! @dario-coscia Is it still possible to include multiple functions in a condition, such as "function=[func1, func2]"? |
No, it is not possible. The kwarg Tell me if it helps. For reference I suggest you Scientific Machine Learning through Physics-Informed Neural Networks: Where we are and What's next, which is a nice and concise introduction to PINNs and Neural Operators. |
Good morning ;) |
@LoveFrootLoops can you send me the snippet of code you are using? I don't understand why you should minimize the same function on the same location. |
Let’s assume I have two PDEs which I want to solve for the same function “u” ont he same domain Omega, let’s call them PDE1 and PDE2. Then I would have to do the following: conditions = { instead of: conditions = { |
I dont really understand. I have never seen that in the same domain more conditions are applied. Can you send me the reference of equations you are dealing with? If it's something useful we can think to make a pull request to make it in the next release. |
Although my demonstration involved PDEs, I want to clarify that my point applies to a broader range of scenarios. For instance, restrictions such as the positivity of 'u' over the domain Omega can be effectively addressed with this approach. However, at present, I must define a new condition for each constraint, even if it pertains to the same domain. Additionally, if I understand correctly, PINA allows for minimizing 'u' given specific input and output data. However, how about handle input and output data that correspond to derivatives of the function 'u.' In this scenario, three conditions would be required over the same domain. It's worth noting that this approach has applications in various fields, including engineering and physics. |
Thank you for the reply. Indeed you are right, it might be useful to insert this possibility. I have opened a new issue #92 for inserting this possibility. Tell me if you are interested working on it. I will close this issue once the tutorials are merged by @ndem0. To follow the discussion please use #92 |
I am new to this GitHub Game but yes sure, I could work on it. |
Upon running the Tutorial 1 Code (copy-pasted), an error message appeared instantly:
The text was updated successfully, but these errors were encountered: