SimVascular / SimVascular/svMultiPhysics

segmentation fault when using FSILS assembly type in Trilinos implementation

Open
#470 1 comment 0 reactions 1 assignee View on GitHub

@dcodoni is already working on this.

Since Nov 10, 2025.

bug
Dominant language
C++
Stars
45
Forks
60
Avg merge
5d 23h
Merged PRs (30d)
11

Description

Description

The FSILS type of assembly when using Trilinos library for linear algebra throw a segmentation fault. This is caused by a variable 'nnzPerRow' which was not defined before. The variable 'nnzPerRow' contains how many non-zero elements there are per row and it is used to assemble the global matrix in Trilinos data structure.

Reproduction

Just set the following:
<Linear_algebra type="trilinos" >
fsils
...
</Linear_algebra>
In the input file. A segmentation fault for invalid memory location will be thrown by the solver.

Expected behavior

The expected result is to use either assembly type ('trilinos' or 'fsils') when selecting Trilinos library for the linear algebra.

Additional context

Even though the default and suggested assembly type is 'trilinos', the 'fsils' type is the only option available whenever 'ustruct' type of equation is selected.

Code of Conduct
  • I agree to follow this project's Code of Conduct and Contributing Guidelines

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.