Skip to content
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

[examples] Add a thermo-elastic example #204

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

hugtalbot
Copy link
Contributor

@hugtalbot hugtalbot commented Oct 8, 2021

Some issues were notice with visualization on SP3 master.
Works fine with release v21.06.00.

Texture visualization with SOFA master does not work due to issue reported here: sofa-framework/sofa#2392

To be merged after #198

thermoelasticity__r60_0003.mp4

@hugtalbot hugtalbot added enhancement New feature or request pr: status to review pr: highlighted in next release Highlight this contribution in the notes of the upcoming release labels Oct 8, 2021
@guparan guparan modified the milestones: v21.06, v21.12 Oct 26, 2021
meca.addObject("SparseLDLSolver", name="LDL LinearSolver", template="CompressedRowSparseMatrixMat3x3d")
meca.addObject("TetrahedronSetTopologyContainer", name="tetContainer", src="@../tetContainer")
meca.addObject("TetrahedronSetGeometryAlgorithms", name="tetGeometry", template="Vec3d")
meca.addObject("MechanicalObject", name="MO", position="@../tetraO.position")
Copy link
Contributor

@damienmarchal damienmarchal Oct 27, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

related to .linkpath and this (PR #265)

@alxbilger
Copy link
Contributor

@hugtalbot Is there something preventing to finish the job?

@hugtalbot
Copy link
Contributor Author

hey @alxbilger you're right, this PR was awaiting the pull-request in SOFA introducing the GeometryType in the mass.
the script should be slightly modified but else it will be possible to merge it.

I do the change and poke you back for a final review 👍

@hugtalbot
Copy link
Contributor Author

Reviews taken into account.

However, this scene often crashes with such error:

double free or corruption (out)

########## SIG 6 - SIGABRT: usually caused by an abort() or assert() ##########

I am suspecting the engine TextureInterpolation

@hugtalbot
Copy link
Contributor Author

[ci-build]

@damienmarchal
Copy link
Contributor

What about this PR, should it be updated or closed or postponed ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pr: highlighted in next release Highlight this contribution in the notes of the upcoming release pr: status wip
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants