Geant4 Installation Issue #137
-
|
Hi, I am having an issue with implementing NCrystal in Geant4. First I followed the INSTALL file and successfully installed it and performed the test successfully. (I made sure to cmake with -DNCRYSTAL_ENABLE_GEANT4=ON). Then, I went into the folder with my Geant4 simulations and did the following: (Note that my file is called DMX instead of exampleapp as in the README)
The error was: Make Error at /data/ljoseph/ncrystalinstall/lib64/cmake/NCrystal/NCrystalConfig.cmake:118 (if): Unknown arguments specified The error seems to imply that the config file (which I have not messed with) has an if command: if ( "GEANT4BINDINGS" IN_LIST NCrystal_FIND_COMPONENTS ) that has unrecognized arguments. I must be doing something wrong as I doubt it is an issue with the NCrystal file. However, I did grep for NCrystal_FIND_COMPONENTS and it was only referenced in NCrystalConfig.cmake so it almost seems undefined as the error suggests. Below I have pasted my full CMakeLists file. Any help is greatly appreciated! We are not sure where to go from here. #---------------------------------------------------------------------------- Setup the projectcmake_minimum_required(VERSION 2.6 FATAL_ERROR) #Logan Added option(WITH_GEANT4_UIVIS "Build example with Geant4 UI and Vis drivers" ON) include(${Geant4_USE_FILE}) if("$ENV{DMXENV_GPS_USE}" AND NOT DEFINED DMXENV_GPS_USE) option(DMXENV_GPS_USE "Build DMX with support for GPS" ${DMXENV_GPS_USE_DEFAULT}) if(DMXENV_GPS_USE) include_directories(${PROJECT_SOURCE_DIR}/include add_executable(DMX DMX.cc ${sources} ${headers}) #Logan Added set(underground_physics_SCRIPTS foreach(_script ${underground_physics_SCRIPTS}) add_custom_target(underground_physics DEPENDS DMX) install(TARGETS DMX DESTINATION bin) |
Beta Was this translation helpful? Give feedback.
Replies: 4 comments 14 replies
-
|
Thanks for the report. What CMake version are you using? |
Beta Was this translation helpful? Give feedback.
-
|
Sorry for the late response, but I am using cmake version 3.10.2 |
Beta Was this translation helpful? Give feedback.
-
|
I think I understand your issue now. I will write more details later after a few more investigattions, but for now if you change the version number in |
Beta Was this translation helpful? Give feedback.
-
|
Hi, I deleted my old comment but this worked! However, now when I am trying to make the example geant4 code by doing "make -j 2 name_of_file", I get the following error In file included from /data/ljoseph/ncrystalinstall/include/NCrystal/NCTypes.hh:28:0, |
Beta Was this translation helpful? Give feedback.
-
|
If easier, here is a screen shot of the first bit of errors |
Beta Was this translation helpful? Give feedback.
-
|
Looks to me like your compiler is not being configured for C++11. Can you provide a tar-ball somewhere with a complete (and minimal!) example of your .cpp and CMakeLists.txt files, and I can see if I can reproduce it? |
Beta Was this translation helpful? Give feedback.
-
|
Im sorry, I thought maybe the Geant4 example on this Github would be good enough. Just to be clear, did you look at the NCrystalTest.tar.gz? I did shave down a lot of unnecessary stuff though, so maybe try this tar below (the one called NCrystalTestSimple.tar.gz). Hopefully this one suffices. I did notice a few new things. I changed project(name) to project(name LANGUAGES CXX) (I also tried project(name LANGUAGES CXX C), as mentioned in the README, and the example file actually maked properly, but running it gave a segmentation fault. I tried this on my other projects and it just gave other errors. However, It doesn't seem like that addition is necessary. Removing it gives the original error I sent you. Also, it might be good to be clear that I am running the right CMAKE command. In my simulation build folder I run cmake -DGeant4_DIR=/data/ljoseph/geant4.10.03.p03-build/geant4-10.3.3 -DNCrystal_DIR=/data/ljoseph/ncrystalinstall /data/ljoseph/Geant4/TN/NCrystalTest where the last path is the path to my simulation. Then I run make -j2 DMX (DMX is the name of my main .cc file). This is correct right? I appreciate all your help so far and I apologize for my struggles, I am still learning. Another thing which might be very helpful if you would be willing to share a CMakeLists file which is written for more modern CMake versions. I inherited these files from an older project and none of us on the team are well-versed enough in CMake to write an updated one, so seeing a CMakeLists example that works would be helpful if you are willing to share one. |
Beta Was this translation helpful? Give feedback.
-
|
So trying your NCrystalTestSimple.tar.gz files, they actually work fine out of the box for me, although my machine might have a different setup that yours. One thing you can try, is to try to get the latest NCrystal instead: v3.7.1 came out yesterday, and include a few fixes inspired by your use-case. In particular, I believe I have fixed the original CMake error, and maybe the error about is_trivially_copyable as well (not 100% sure, since I never had that error in the first place on any of our tested systems). Although it won't help you right now, I should mention that my plans for the fall include revisiting the NCrystal-Geant4 hooks, and hopefully will also end up in better documentation, recipes, examples, ... Cheers, |
Beta Was this translation helpful? Give feedback.
-
|
Thanks for all your help! It seems like this patch is better. While I am still having some errors in compiling my code (which works in base Geant4), the example code on this github is now fully compiling. However, running it gives a segmentation fault and I discovered that it is faulting at the line which creates the physics lists: Kindly, |
Beta Was this translation helpful? Give feedback.
-
|
To answer the first half, I can only guess. Perhaps you have an incomplete geant4-datafile setup which (perhaps) only triggers with an HP physics list? Otherwise, I was using geant4-10.04.p03 to test, which differs from yours. For the second question, note that I am currently working on better Geant4-NCrystal hooks which will also result in updated documentation. But as a quick example, here is a modern (aka "cmake3" or "target-based") cmake lists which works for me: Note that you get the geant4 dependency (with link and compilation flags) injected automatically when linking against the |
Beta Was this translation helpful? Give feedback.
-
|
Hi Thomas, We finally got the example code to run! Thank you so much for your help I doubt we could have gotten it to work otherwise - we really appreciate it. My code is still not working but its clear that that is my issue now and not NCrystal. |
Beta Was this translation helpful? Give feedback.
-
|
Good to hear that you at least managed to get past this hurdle! And hopefully usage of Geant4+NCrystal will become a lot more streamlined later this fall, with better documentation etc. |
Beta Was this translation helpful? Give feedback.
-
|
And please also provide information about your system (in particular compiler version, how you installed it, etc.). |
Beta Was this translation helpful? Give feedback.
-
|
Hi, here is a screenshot I took of stuff I thought was relevant I am still new to linux and have gotten help from others in my group. As a result I do not know exactly how it was installed but if necessary I can search around for who knows. In summary though, gcc version is 12.2, python is 3.10.8, and the linux release is 7.9, I am also using python through conda |
Beta Was this translation helpful? Give feedback.
-
|
So one thing that strikes me here is that your are using python through conda, but a compiler which is not provided by conda. Such a mix is most likely not ideal. |
Beta Was this translation helpful? Give feedback.
-
|
Right, this is due to how the remote computer is setup, I am working on getting python downloaded so I can access it not through Conda. |
Beta Was this translation helpful? Give feedback.
To answer the first half, I can only guess. Perhaps you have an incomplete geant4-datafile setup which (perhaps) only triggers with an HP physics list? Otherwise, I was using geant4-10.04.p03 to test, which differs from yours.
For the second question, note that I am currently working on better Geant4-NCrystal hooks which will also result in updated documentation. But as a quick example, here is a modern (aka "cmake3" or "target-based") cmake lists which works for me: