nikhilgupta10 / GridLAB-D

Other
1 stars 0 forks source link

#742 Verify Matlab link works correctly, #2480

Closed nikhilgupta10 closed 7 years ago

nikhilgupta10 commented 7 years ago

There have been issues in the past with Matlab compiled code being finicky about switching of Matlab versions. Verify that v3.0 will compile (1) without Matlab on the machine, (2) with various versions of Matlab. Work with build server to ensure the automated build process reflects this.

,

nikhilgupta10 commented 7 years ago

nikhilgupta10 imported these comments from Sourceforge: "andyfisher": * owner changed from andyfisher to ftuffner

I can't seem to get the link to work. I've modified config_extern.h so that no MATLAB was found and it built fine and no validation errors. But when I add back in all the definitions in config_extern.h to build with MATLAB it also built but I ran test_avi.glm and it failed out with this message.

ERROR [INIT] : link '../matlab_avi.link' failed: file open failed
C:\Projects\GridLAB-D_Builds\V3.0\core\link\matlab\autotest-disabled\test_avi.glm(12): unable to link '../matlab_avi.link'
C:\Projects\GridLAB-D_Builds\V3.0\core\link\matlab\autotest-disabled\test_avi.glm(12): load failed at or near 'link ../matl...'

I can get the file to run fine in my x64 build of trunk(I can only get this to work in x64 because I have a 64 bit version of MATLAB R2011a). Also in addition to having those MATLAB definitions in config_extern.h I had to follow the steps on the wiki to set up the link correctly. Those steps involved adding some paths to the properties or glxmatlab which we removed from 3.0. So I'm confused how this is supposed to work properly for 3.0. It's like config_external.h isn't editing the matlab.vcproj file appropriately.

,

"ftuffner": * status changed from assigned to accepted

Applied changes to configuration script in r4350/r4351 to resolve x86 vs. x64 issue. However, if both reside on the machine, egistering\ each version is often necessary when swapping builds. This will need to be documented.

,

"ftuffner": * status changed from accepted to closed

MATLAB detection is handled via configure.bat script now. Appears to work properly on all build servers and runs.

,