Skip to content

Add missing plugin checks in python scenes#78

Closed
fredroy wants to merge 2 commits intomasterfrom
ignore_py_cuda_scenes
Closed

Add missing plugin checks in python scenes#78
fredroy wants to merge 2 commits intomasterfrom
ignore_py_cuda_scenes

Conversation

@fredroy
Copy link
Contributor

@fredroy fredroy commented Mar 13, 2026

The missing-plugin filter checks was only searching for xml syntax

Co-authored-by: Paul Baksic <30337881+bakpaul@users.noreply.github.com>
Copy link
Contributor

@bakpaul bakpaul left a comment

Choose a reason for hiding this comment

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

THis doesn't work because python allows to use a variable as input of the pluginNames. Breaking this script. Using this solution it deactivates more than a 100 scenes on the CI. SO I would advocate to remove the scenes using SofaCuda explicitly in the .scene_tests file. I'll do it. In the meantime I close this PR

--> see sofa-framework/sofa#6014

@bakpaul bakpaul closed this Mar 13, 2026
@fredroy fredroy deleted the ignore_py_cuda_scenes branch March 13, 2026 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants