Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge SlicerFreeSurfer and SlicerFreeSurferCommands and move into the Slicer GitHub organization? #15

Open
benzwick opened this issue Jun 13, 2023 · 3 comments

Comments

@benzwick
Copy link

benzwick commented Jun 13, 2023

Should we merge SlicerFreeSurfer and SlicerFreeSurferCommands, and move them to the Slicer GitHub organization?

Please see the discussion here for more details.

One thing to consider is that the SlicerFreeSurfer extension doesn't have any external dependencies as far as I'm aware, whereas the SlicerFreeSurferCommands extension depends on FreeSurfer which needs to be installed and configured by the user, takes up about 14 GB of disk space, and can only be installed on Linux/Unix OS (on Windows WSL can be used).

@benzwick benzwick changed the title Merge SlicerFreeSurfer and SlicerFreeSurferCommands and move into the Slicer GitHub organization Merge SlicerFreeSurfer and SlicerFreeSurferCommands and move into the Slicer GitHub organization? Jun 14, 2023
@jcfr
Copy link
Contributor

jcfr commented Aug 16, 2023

Instead moving the project SlicerFreeSurferCommands to the PerkLab GitHub organization, per suggestion of @pieper , the idea would be the following:

  • bundle the modules inside the PerkLab/SlicerFreeSurfer extension
  • make it clear that these commands are only available if freesurfer is installed.

That is, make it clear to the user at runtime, rather than in documentation, what features would be available if freesurfer is installed along with pointers to the installation instructions (even a button that opens the web page).

@Sunderlandkyl
Copy link
Contributor

I agree, adding the SlicerFreeSurferCommands to SlicerFreeSurfer sounds like the best approach.

@lassoan
Copy link
Contributor

lassoan commented Dec 3, 2024

@benzwick would you mind creating a pull request that adds module from https://github.com/SlicerCBM/SlicerFreeSurferCommands to this extension?

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

No branches or pull requests

4 participants