View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0003823 | Slicer4 | Extension: SlicerRT / Module: Plastimatch | public | 2014-08-28 15:13 | 2016-09-30 11:30 |
Reporter | lassoan | Assigned To | gregsharp | ||
Priority | normal | Severity | major | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | |||||
Target Version | Fixed in Version | Slicer 4.5.0-1 | |||
Summary | 0003823: Plastimatch and dependent modules cannot be loaded on windows 8 | ||||
Description | Installed the latest nightly build of Slicer and SlicerRT extension. Lots of modules are missing (all that rely on Plastimatch) and the following is logged by Slicer: [CRITICAL][Qt] 28.08.2014 17:30:31 [] (unknown:0) - Error(s): Logged the loading process with sxstrace and got this: ERROR: Cannot resolve reference Microsoft.VC90.OpenMP,processorArchitecture="amd64",publicKeyToken="1fc8b3b9a1e18e3b",type="win32",version="9.0.21022.8". So, it seems that the installation package does not contain Microsoft.VC90.OpenMP. Installing VC2008 redistributable package would probably make the problem go away, but I'm not sure users could figure this out. Either CMake (more precisely: InstallRequiredSystemLibraries.cmake) should be extended or the missing DLL should be added to the Slicer CMake files. | ||||
Tags | No tags attached. | ||||
Hmm, yes I see. This is a problem. I wonder why it never came up before. For a short term, I will disable OpenMP for Windows build. |
|
It did not come up before because the packaging was broken before so that not the right redistributable DLLs were included in Windows packages. Therefore if somebody did not have VS2008 redistributable package installed, then Slicer did not start. Now DLLs that are required by Slicer core are correctly included, so Slicer starts, but OpenMP DLL is not included, so SlicerRT can have problems even if Slicer start correctly. |
|
Workaround committed, not yet fully tested. https://www.assembla.com/code/dG15GuCs4r4l4UeJe5cbCb/ctGUxGF88r4yH5acwqjQXA/commit/1945 Andras, I agree with your analysis. Good to fix InstallRequiredSystemLibraries.cmake. I will prepare a patch. |
|
Bug report to CMake. |
|
Patch submitted to cmake mantis and email list. http://www.cmake.org/pipermail/cmake/2014-August/058439.html |
|
Patch accepted. Will roll with CMake 3.1. |
|
Slicer will need a little patch as well. |
|
Fixed in r24125 |
|
The issue reappeared, see |
|
Fixed. https://app.assembla.com/spaces/slicerrt/tickets/664-plastimatch-openmp-disabled-on-windows/details# |
|
Fixed https://app.assembla.com/spaces/slicerrt/tickets/664-plastimatch-openmp-disabled-on-windows/details# |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2014-08-28 15:13 | lassoan | New Issue | |
2014-08-28 15:13 | lassoan | Status | new => assigned |
2014-08-28 15:13 | lassoan | Assigned To | => gregsharp |
2014-08-29 07:50 | gregsharp | Note Added: 0012443 | |
2014-08-29 07:58 | lassoan | Note Added: 0012444 | |
2014-08-29 08:57 | gregsharp | Note Added: 0012445 | |
2014-08-29 12:53 | gregsharp | Note Added: 0012446 | |
2014-08-29 13:54 | gregsharp | Note Added: 0012447 | |
2014-09-02 09:01 | gregsharp | Note Added: 0012448 | |
2014-09-02 09:50 | gregsharp | Note Added: 0012449 | |
2015-11-02 15:04 | jcfr | Note Added: 0013512 | |
2015-11-02 15:04 | jcfr | Status | assigned => resolved |
2015-11-02 15:04 | jcfr | Fixed in Version | => Slicer 4.5.0-1 |
2015-11-02 15:04 | jcfr | Resolution | open => fixed |
2016-04-13 14:39 | pinter | Note Added: 0013857 | |
2016-04-13 14:39 | pinter | Status | resolved => confirmed |
2016-09-30 11:29 | gregsharp | Note Added: 0014136 | |
2016-09-30 11:30 | gregsharp | Note Added: 0014137 | |
2016-09-30 11:30 | gregsharp | Status | confirmed => closed |