svalinn / DAGMC

Direct Accelerated Geometry Monte Carlo Toolkit
https://svalinn.github.io/DAGMC
Other
96 stars 61 forks source link

Build error #900

Closed Coperbytes closed 11 months ago

Coperbytes commented 11 months ago

Describe the Bug
While making the build does not find the function find_volumme

To Reproduce
Specify the necessary steps to reproduce the behavior. For example, supply the cmake command used for building if there is a build error, or the steps taken when using the DAGMC that produced the error.

Expected Behavior
Should do clean build

Screenshots or Code Snippets

/openmc/DAGMC/DAGMC/src/dagmc/DagMC.cpp: In member function ‘moab::ErrorCode moab::DagMC::find_volume(const double*, moab::EntityHandle&, const double*)’:
/openmc/DAGMC/DAGMC/src/dagmc/DagMC.cpp:818:32: error: ‘class double_down::RayTracingInterface’ has no member named ‘find_volume’
  818 |   ErrorCode rval = ray_tracer->find_volume(xyz, volume, uvw);
      |                                ^~~~~~~~~~~
make[2]: *** [src/dagmc/CMakeFiles/dagmc-shared.dir/build.make:76: src/dagmc/CMakeFiles/dagmc-shared.dir/DagMC.cpp.o] Error 1

Please complete the following information regarding your system:

gonuke commented 11 months ago

Thanks for making an issue about this. I think @pshriwise and @ebknudsen are working on a fix for this.

gonuke commented 11 months ago

In the meantime, if you build DAGMC without Double Down, it should work correctly.

ebknudsen commented 11 months ago

@pshriwise fixed this in the develop-branch with this PR: https://github.com/pshriwise/double-down/pull/42. It has however not been merged into the main branch. Did you build double-down from the main branch @Coperbytes? Or use an older build?

gonuke commented 11 months ago

I think it's merged but without a release yet...?

ebknudsen commented 11 months ago

Nope - it is in develop though. main hasn't been touched for 2 yrs unfortunately. A new release would prob. be a good thing.

ebknudsen commented 11 months ago

I mean to say - the fix was merged into the develop-branch, but not into main

pshriwise commented 11 months ago

Agreed on the double-down release, @ebknudsen. I'll try to find some time to work on that soon.

Coperbytes commented 11 months ago

I mean to say - the fix was merged into the develop-branch, but not into main

@ebknudsen I used installed scripts from your other repo openmc_install_scripts

This is the branch that was downloaded from the script

 ~/openmc/DAGMC/DAGMC/ git info

## Remote URLs:

origin  https://github.com/svalinn/DAGMC.git (fetch)
origin  https://github.com/svalinn/DAGMC.git (push)

## Remote Branches:

  origin/HEAD -> origin/develop
  origin/develop

## Local Branches:

* develop

## Submodule(s):

 36a6b4b src/pyne/pyne (0.7.7-34-g36a6b4bf)

## Most Recent Commit:

commit b3e2b132aef7d4bf333af4ee9999c0c2ec752601
Author: Patrick Shriwise 

    Merge pull request #869 from gonuke/doc_review_2023_03
ebknudsen commented 11 months ago

@Coperbytes Sorry - me bad - I meant which branch of double-down. But if you use the openmc_install_scripts then It ought to have been develop for double-down as well. Which platform are you on?

Coperbytes commented 11 months ago

I am installing on Ubuntu 22.04, the double_down-install.sh script has main instead of develop for installation

git clone --single-branch --branch main --depth 1 https://github.com/pshriwise/double-down.git

by changing to develop branch it fixed the build error, now I have issue in installation of openmc should I close this issue and create new issue in openmc_install_scripts repo?

openmc/openmc/tests/cpp_unit_tests/test_file_utils.cpp:2:10: fatal error: catch2/catch_test_macros.hpp: No such file or directory
    2 | #include <catch2/catch_test_macros.hpp>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make[2]: *** [tests/cpp_unit_tests/CMakeFiles/test_file_utils.dir/build.make:76: tests/cpp_unit_tests/CMakeFiles/test_file_utils.dir/test_file_utils.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:994: tests/cpp_unit_tests/CMakeFiles/test_file_utils.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
/home/kp/openmc/openmc/tests/cpp_unit_tests/test_distribution.cpp:3:10: fatal error: catch2/catch_test_macros.hpp: No such file or directory
    3 | #include <catch2/catch_test_macros.hpp>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make[2]: *** [tests/cpp_unit_tests/CMakeFiles/test_distribution.dir/build.make:76: tests/cpp_unit_tests/CMakeFiles/test_distribution.dir/test_distribution.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:968: tests/cpp_unit_tests/CMakeFiles/test_distribution.dir/all] Error 2
/home/kp/openmc/openmc/tests/cpp_unit_tests/test_tally.cpp:2:10: fatal error: catch2/catch_test_macros.hpp: No such file or directory
    2 | #include <catch2/catch_test_macros.hpp>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make[2]: *** [tests/cpp_unit_tests/CMakeFiles/test_tally.dir/build.make:76: tests/cpp_unit_tests/CMakeFiles/test_tally.dir/test_tally.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:1020: tests/cpp_unit_tests/CMakeFiles/test_tally.dir/all] Error 2
[ 97%] Linking CXX executable bin/openmc
[ 97%] Built target openmc
make: *** [Makefile:146: all] Error 2
ebknudsen commented 11 months ago

@Coperbytes please do - that is where it belongs - I'll see if I can't fix that later today