Skip to content

Conversation

@bakpaul
Copy link
Contributor

@bakpaul bakpaul commented Jan 21, 2026

Metis crashes on the release on windows for some scene (e.g. examples/Demos/SofaScene.scn), so we need to avoid auto-loading it on windows.

This PR is only on the release branch in hope that we fix this for the next release.


By submitting this pull request, I acknowledge that
I have read, understand, and agree SOFA Developer Certificate of Origin (DCO).


Reviewers will merge this pull-request only if

  • it builds with SUCCESS for all platforms on the CI.
  • it does not generate new warnings.
  • it does not generate new unit test failures.
  • it does not generate new scene test failures.
  • it does not break API compatibility.
  • it is more than 1 week old (or has fast-merge label).

@bakpaul bakpaul added pr: fix Fix a bug pr: status to review To notify reviewers to review this pull-request topic for next dev-meeting PR to be discussed in sofa-dev meeting labels Jan 21, 2026
@hugtalbot
Copy link
Contributor

hugtalbot commented Jan 22, 2026

  • Couple of scenes are crashing due to metis, only occurring on Windows. It might take time to fix it, that's the reason why the suggestion is to remove it from autoload
  • The bug does not occur with Pixi
  • Ideas to investigate
    • Test binaries from Pixi and investigate the Pixi patch
    • Try the release without CGAL to see if it solves the issue
  • OrderingMethod for a direct solver is a property hidden to user, which is based on a plugin (Metis) by default if one is found. We could find an alternative Scotch

@hugtalbot hugtalbot removed the topic for next dev-meeting PR to be discussed in sofa-dev meeting label Jan 23, 2026
@hugtalbot hugtalbot marked this pull request as draft January 23, 2026 08:32
@olivier-roussel
Copy link
Contributor

olivier-roussel commented Jan 23, 2026

I could do further testing and I think the problem comes from the new version of metis (v5.2.1).
My working Pixi build was older than the metis version bump (#5732). But I could reproduce the bug with fetched metis v5.2.1 from our fork. Scenes run fine with metis v5.1.0.

@olivier-roussel
Copy link
Contributor

The scenes also run fine when using the metis v5.2.1 package from conda-forge (warning: the current one is broken on Windows, a fix is in progress on conda-forge conda-forge/metis-feedstock#47).
Even if we do not want to use the conda package all the time, this is good news as it could be due to one of the upstream patches of the conda forge package on v5.2.1, which we could integrate in our fork for v5.2.1.
I'll continue investigate on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: fix Fix a bug pr: status to review To notify reviewers to review this pull-request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants