Element deletion using right mouse click resulted in segfault

[CherryJGit]

This is the scn file I used:
https://github.com/sofa-framework/sofa/blob/master/examples/Component/SolidMechanics/FEM/HexahedralFEMForceField.scn
The msh file among them is:
https://github.com/sofa-framework/sofa/blob/master/share/mesh/nine_hexa.msh
I found that when this document is running and the ablation effect is carried out, that is, pressing Shift + the right mouse button, it will result in the following error log.Is it that my runsofa is lacking plugins, or is this scn file incomplete.I hope to get your help.

########## SIG 11 - SIGSEGV: segfault ##########
sofa::helper::BackTrace::dump
sofa::helper::BackTrace::sig
seh_filter_exe
seh_filter_exe
_C_specific_handler
_chkstk
RtlRaiseException
KiUserExceptionDispatcher
sofa::component::topology::container::dynamic::TriangleSetTopologyModifier::removeEdgesProcess
sofa::component::topology::container::dynamic::TriangleSetTopologyModifier::removeTrianglesProcess
sofa::component::topology::container::dynamic::TriangleSetTopologyModifier::removeTriangles
sofa::component::topology::mapping::Quad2TriangleTopologicalMapping::updateTopologicalMappingTopDown
sofa::simulation::TopologyChangeVisitor::processNodeTopDown
sofa::simulation::graph::DAGNode::executeVisitorTopDown
sofa::simulation::graph::DAGNode::executeVisitorTopDown
sofa::simulation::graph::DAGNode::doExecuteVisitor
sofa::simulation::Node::executeVisitor
sofa::component::topology::container::dynamic::PointSetTopologyModifier::propagateTopologicalChanges
sofa::component::topology::container::dynamic::QuadSetTopologyModifier::removeQuads
sofa::component::topology::mapping::Hexa2QuadTopologicalMapping::updateTopologicalMappingTopDown
sofa::simulation::TopologyChangeVisitor::processNodeTopDown
sofa::simulation::graph::DAGNode::executeVisitorTopDown
sofa::simulation::graph::DAGNode::executeVisitorTopDown
sofa::simulation::graph::DAGNode::doExecuteVisitor
sofa::simulation::Node::executeVisitor
sofa::component::topology::container::dynamic::PointSetTopologyModifier::propagateTopologicalChanges
sofa::component::topology::container::dynamic::HexahedronSetTopologyModifier::removeHexahedraProcess
sofa::component::topology::container::dynamic::HexahedronSetTopologyModifier::removeHexahedra
sofa::gui::component::performer::TopologicalChangeManager::removeItemsFromTriangleModel
sofa::gui::component::performer::TopologicalChangeManager::removeItemsFromCollisionModel
sofa::gui::component::performer::TopologicalChangeManager::removeItemsFromCollisionModel
sofa::gui::component::performer::RemovePrimitivePerformer<sofa::defaulttype::StdVectorTypes<sofa::type::Vec<3,double>,sofa::type::Vec<3,double>,double> >::execute
sofa::gui::component::performer::BaseMouseInteractor::updatePosition
sofa::simulation::BehaviorUpdatePositionVisitor::processNodeTopDown
sofa::simulation::graph::DAGNode::executeVisitorTopDown
sofa::simulation::graph::DAGNode::executeVisitorTopDown
sofa::simulation::graph::DAGNode::executeVisitorTopDown
sofa::simulation::graph::DAGNode::doExecuteVisitor
sofa::simulation::Node::executeVisitor
sofa::simulation::DefaultAnimationLoop::step
sofa::simulation::Simulation::animate
sofa::gui::qt::RealGUI::step
QMetaObject::activate
QTimer::timerEvent
QObject::event
QApplicationPrivate::notify_helper
QApplication::notify
QCoreApplication::notifyInternal2
QEventDispatcherWin32::event
QApplicationPrivate::notify_helper
QApplication::notify
QCoreApplication::notifyInternal2
QCoreApplicationPrivate::sendPostedEvents
qt_plugin_query_metadata
QEventDispatcherWin32::processEvents
CallWindowProcW
DispatchMessageW
QEventDispatcherWin32::processEvents
qt_plugin_query_metadata
QEventLoop::exec
QCoreApplication::exec
sofa::gui::qt::RealGUI::mainLoop
sofa::gui::common::GUIManager::MainLoop
sofa::gui::common::GUIManager::MainLoop
sofa::gui::common::GUIManager::MainLoop
BaseThreadInitThunk
RtlUserThreadStart
[WARNING] [Sofa.Simulation.Graph] the library has not been cleaned up (sofa::simulation::graph::cleanup() has never been called, see sofa/helper/init.h)
[WARNING] [SofaSimulationCommon] the library has not been cleaned up (sofa::simulation::common::cleanup() has never been called, see sofa/helper/init.h)

[bakpaul]

Hello !

Thank you for reporting this crash. I am not the biggest specialist of topological changes. Let's ask teh actual ones : @epernod do you think that the TopologicalMapping is in fault here ? @hugtalbot have you any idea ?

[hugtalbot]

The problem comes from the VisualModel which is not supporting topological change.
I modified the example so that it works unconditionally with or without topological changes : #5367

Thanks @CherryJGit

[CherryJGit]

I have updated and verified the examples. Thank you.

[epernod]

#5367 has been updated to handle the change. Check the PR for the bug description. Here is the result:

SofaGLFW.-.HexahedralFEMForceField.scn.2025-03-27.21-59-12.-.0.00.05-0.00.15.mp4

[CherryJGit]

I have tested it, and the result is the same as yours. Thank you for your help.