Skip to content

Slicer core sdk based#4

Merged
Thibault-Pelletier merged 6 commits intomainfrom
slicer-core-sdk-based
Mar 18, 2026
Merged

Slicer core sdk based#4
Thibault-Pelletier merged 6 commits intomainfrom
slicer-core-sdk-based

Conversation

@AlexyPellegrini
Copy link
Collaborator

This PR contains required changes for the future slicer-core support.
It makes the project buildable using the slicer-core-sdk.

@Thibault-Pelletier Thibault-Pelletier changed the base branch from main to slicer-wheel-support February 19, 2026 08:59
@AlexyPellegrini AlexyPellegrini changed the base branch from slicer-wheel-support to main March 16, 2026 15:00
@AlexyPellegrini AlexyPellegrini force-pushed the slicer-core-sdk-based branch 2 times, most recently from a10031c to 85e8618 Compare March 16, 2026 15:37
@AlexyPellegrini
Copy link
Collaborator Author

@Thibault-Pelletier ready

@AlexyPellegrini AlexyPellegrini force-pushed the slicer-core-sdk-based branch 3 times, most recently from 4cdffec to 6856316 Compare March 17, 2026 15:50
@AlexyPellegrini AlexyPellegrini force-pushed the slicer-core-sdk-based branch 2 times, most recently from 3c04dcc to 16fa790 Compare March 18, 2026 08:50
Thibault-Pelletier and others added 6 commits March 18, 2026 09:53
- Convert the Logic, MRML and MRMLDM modules to vtk.module
- Add flag for VTK module auto init
- Ensure extension and wheel builds works with a shared code base
- CI job does a single wheel build on Linux, Mac and Windows
- Release job uses semantic release, build all wheels and upload to pypi
Could not find a proper way to apply generated patch.
Doing the following:
- `"build command" 2> report.txt`
- `fix_include < report.txt`
fails due to remove and add includes overlapping.
@Thibault-Pelletier
Copy link
Collaborator

LGTM!

@Thibault-Pelletier Thibault-Pelletier merged commit 922a87c into main Mar 18, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants