-
Notifications
You must be signed in to change notification settings - Fork 54
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
Repeated variable name in Hyperelasticity demo #117
Comments
For point 1: one should probably rename the second variable to residual
This is a bug in pyvista If you use the VTXWriter to visualize it with Paraview it renders properly. |
Thank you for your quick response! Another request; might not be your priority right now, but adding problem description (or a link to the original problem) to this tutorial might help. Anyway, your tutorials are fantastic and helping me to learn FEniCSx fairly quickly. |
Do you mean a description of the setup for the hyperelasticity code? |
Geometry, material properties, the governing equation, and the boundary condition. Most of it is generally very clear for someone in the computational mechanics. It took me some time to understand that traction is being applied later for the deflection as initially it was set to zero. |
Hyperelasticity demo uses F as deformation gradient and then again as the left hand side of the nonlinear variational problem. While the code works fine; not sure how fem.petsc.NonlinearProblem(F, u, bcs) identifying the right F variable? I would probably rename one of the variables.
Also in the same example, the PyVista plot shows tetrahedron while rendering the plot for 2nd order Lagrange. Is it a bug from PyVista or VTK? Any idea if it would render properly on ParaView or some other package?
The text was updated successfully, but these errors were encountered: