EGSnrc icon indicating copy to clipboard operation
EGSnrc copied to clipboard

IAEA phase space files compiled after addphsp

Open ojalaj opened this issue 8 years ago • 10 comments

With the develop branch of EGSnrc2016 from April 1st I get the following error message when using addphsp to concatenate the IAEA phase space files from my test simulations:

Sorry, this code has not been compiled with the IAEA phase space handling macros.

The distribution is installed on a Linux cluster (Scientific Linux 6.7) as per the GitHub Wiki instructions with gfortran/gcc/g++/make using the shell script with default options (I'm more than happy to send the configuration log files, if requested). The only modifications that have been made after the installation are in MAXFIELD and MAX_POINT parameter values of several CMs and increasing IMAX/JMAX/KMAX/CTKMAX/MXMED values for ctcreate and dosxyznrc user codes. Otherwise the code is unchanged. I have encountered the problem with every test simulation that I have run, even with EX16MVp example code. The simulations themselves run fine.

ojalaj avatar Apr 07 '16 09:04 ojalaj

Try going into $OMEGA_HOME/progs/addphsp and re-make addphsp (you may have to remove the .f file to force the recompile). There's a bug in the installation script: the iaea phase space libraries are currently compiled AFTER addphsp.

blakewalters avatar Apr 07 '16 13:04 blakewalters

Thanks @blakewalters - your suggestion worked. I've always thought that make clean prior to make would do the job, but obviously it is not enough.

ojalaj avatar Apr 07 '16 17:04 ojalaj

I also ran into this issue. I had to re-make beamdp for use with IAEA phase space files too.

AndrewWAlexander avatar Sep 02 '16 04:09 AndrewWAlexander

@ojalaj there are a few make scripts in the code base where I've noticed make clean is not defined. I should probably catalog and submit those as bugs. Statdose is definitely one and I think addphsp is one too.

crcrewso avatar Sep 02 '16 15:09 crcrewso

@blakewalters indeed please submit this lack of make clean as a new issue, and let's close this older one.

ftessier avatar Sep 02 '16 16:09 ftessier

Was the order of compilation of the IAEA libraries and addphsp ever fixed? Because I recently installed the latest develop branch and it seems like I still had to make clean and re-compile addphsp to get IAEA support.

mchamberland avatar Sep 13 '17 03:09 mchamberland

Noticed the same thing!

ojalaj avatar Sep 13 '17 06:09 ojalaj

Looking at the configure script, it does look like the IAEA stuff comes after addphsp is compiled. @ftessier I suggest re-opening this issue.

mchamberland avatar Sep 13 '17 16:09 mchamberland

The original issue has not been resolved: the IAEA phase space file library should be compiled before the addphsp application.

ftessier avatar Sep 14 '17 12:09 ftessier

The reason IAEA is compiled after addphsp and friends is that the configure script deals with Fortran and C compilation, and then calls the configure_c++ script to handles all the C++ configuration and compilation (IAEA library is c++). I have re-written a much cleaner version of the configure script where everything is integrated together, properly POSIX and documented, etc.; but left that hanging years ago. I will unearth to see if it fixes this issue...

ftessier avatar Apr 22 '21 23:04 ftessier