Andras Lasso

Results 869 comments of Andras Lasso

I suspect that Slicer_*_EXTRA_CMAKE_ARGS would be too low-level access to be exposed (it would be difficult to make it future-proof, it could interfere with options set using higher-level APIs, it...

Good catch! Yes, we are moving away from the wiki, with all documentation and data hosted in extension repositories, so we should use the images stored in the extension's repository.

The main issue was that icon downloading did not follow redirects. The reworked extensions manager now follows redirects, so only a handful of extensions are without icons - all due...

Agreed. I've switched to a fork for CleverSegmentation and BreastImplantAnalyzer. @jamesobutler Could you merge RegularizedFastMarching, ModelClip, and BrainVolumeRefinement patches into your forks' main branches so that I can switch to...

> The plan would also be that if an extension or other code specifically calls to create a vtkMRMLMarkupsFiducialNode that they would be met with a deprecation message that would...

Note that this is already implemented in extension wizard: https://github.com/Slicer/Slicer/blob/master/Modules/Scripted/ExtensionWizard/ExtensionWizard.py#L320-L391 Loading individual modules manually is not ideal: - you would need to resolve dependencies between modules manually to load them...

> Yes, I looked at the ExtensionWizard for ideas, but it's mixed with the gui there and not reusable. Moving the logic of it to slicer.util would make sense. Fully...

LoadModule left as public by accident (or as a temporary workaround) see comments in the source files for details. I'm perfectly fine to expose features that don't always work to...

Just to clarify, I'm fine if this change is merged as is. It is just an experimental feature anyway. Just brought up the points above to give a chance to...

What software you are trying to load the RTSTRUCT into? Does that software can read RTSTRUCT but not those RTSTRUCTs that Slicer exported?