View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0002363Slicer4Core: Base Codepublic2012-07-27 09:222018-03-02 11:00
Reporterdemian Assigned Tojcfr  
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionwon't fix 
Product Version 
Target VersionSlicer 4.7.0Fixed in Version 
Summary0002363: Add support for loading directory from command line
Description

From Demian:

I have to comments:
1) What happens if I want to load fiber bundles? Now they are loaded as a Model by default
2) The directory issue

The way slicer 3 used to solve this is that it would open the "Add data into the scene" window with the rows corresponding to the files that were loaded. The thing is that now, for the volumes you can tell slicer to center them or take them as a label volume post-loading (which before was impossible).

So for the fiber bundles, maybe a solution is to first try to load the model as a fiber bundle (which is a polydata that contains lines only) and then try for the other model types?

Cheers
D

TagsNo tags attached.

Relationships

related to 0001594 closedcrmullin specifying scene or data to load on the command line 
related to 0002364 closedjcfr Refine how "fiber bundles" are loaded from command line 

Activities

crmullin

crmullin

2013-01-03 12:21

developer   ~0007607

Here is an almost working example: https://github.com/chrismullins/Slicer/commit/96ff9d9468be0f4d1825dad4fa54ae5a53608d3d

This commit induces a crash when a .mrml scene is loaded, which is why I won't integrate it yet - but I believe it is an overall more elegant solution than what we have in Slicer currently.
crmullin

crmullin

2013-01-04 11:08

developer   ~0007612

Fixed here: https://github.com/chrismullins/Slicer/commit/e889b5ee436d59ae4c85e7c0e5894be59ac337cd

Awaiting code review, but feel free to try this out.
crmullin

crmullin

2013-01-09 12:34

developer   ~0007638

Here's the latest version. Now it looks at all the unparsed arguments, first determining which could are file paths, and only tries to load them. This allows one to give the command $ ./Slicer --attach-process /path/to/SlicerScene.mrml

https://github.com/chrismullins/Slicer/commit/5fa13ce53e47e2b22b647589436261fc5c3af83b
crmullin

crmullin

2013-01-23 14:52

developer   ~0007727

Jc warns to make sure that .py files are excluded. Also we have some ideas for a more robust file loading scheme in the future - but it will be a bigger project.

Demian, if you are still having this issue feel free to integrate this patch from my branch and let me know what you think.
lassoan

lassoan

2017-09-27 14:31

developer   ~0015200

There was no feedback from the user that it was still an issue.

Issue History

Date Modified Username Field Change
2012-07-27 09:22 jcfr New Issue
2012-07-27 09:22 jcfr Status new => assigned
2012-07-27 09:22 jcfr Assigned To => pieper
2012-07-27 09:23 jcfr Reporter jcfr => demian
2012-07-27 09:23 jcfr Assigned To pieper => crmullin
2012-07-27 09:23 jcfr Target Version => Slicer 4.3.0
2012-07-27 09:23 jcfr Relationship added related to 0001594
2013-01-03 12:21 crmullin Note Added: 0007607
2013-01-04 11:08 crmullin Note Added: 0007612
2013-01-09 12:34 crmullin Note Added: 0007638
2013-01-23 14:52 crmullin Note Added: 0007727
2013-09-03 05:28 jcfr Target Version Slicer 4.3.0 => Slicer 4.4.0
2014-05-13 10:00 jcfr Assigned To crmullin => jcfr
2014-07-29 10:09 jcfr Target Version Slicer 4.4.0 => Slicer 4.5.0-1
2015-11-02 05:07 jcfr Target Version Slicer 4.5.0-1 => Slicer 4.5.1
2015-11-02 08:52 jcfr Relationship added related to 0002364
2016-10-12 02:56 jcfr Target Version Slicer 4.5.1 => Slicer 4.7.0
2017-09-27 14:31 lassoan Status assigned => resolved
2017-09-27 14:31 lassoan Resolution open => won't fix
2017-09-27 14:31 lassoan Note Added: 0015200
2018-03-02 11:00 jcfr Status resolved => closed