Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Skeleton refactoring #28

Merged
merged 77 commits into from
Dec 9, 2022
Merged

Skeleton refactoring #28

merged 77 commits into from
Dec 9, 2022

Conversation

massimim
Copy link
Collaborator

@massimim massimim commented Oct 25, 2022

New skeleton implementation.

Unit test results on Snoopy (DGX A100)

Create new tag: 20221125-2159 - Experimental
Cannot find file: /home/max/repos/Neon/temp/Neon/build/DartConfiguration.tcl
Test project /home/max/repos/Neon/temp/Neon/build
      Start  1: coreUt_chrono
 1/43 Test  #1: coreUt_chrono .....................   Passed   36.00 sec
      Start  2: coreUt_cli
 2/43 Test  #2: coreUt_cli ........................   Passed    0.00 sec
      Start  3: coreUt_digraph
 3/43 Test  #3: coreUt_digraph ....................   Passed    0.00 sec
      Start  4: coreUt_exceptions
 4/43 Test  #4: coreUt_exceptions .................   Passed    0.00 sec
      Start  5: coreUt_io
 5/43 Test  #5: coreUt_io .........................   Passed    9.52 sec
      Start  6: coreUt_logging
 6/43 Test  #6: coreUt_logging ....................   Passed    0.00 sec
      Start  7: coreUt_tools
 7/43 Test  #7: coreUt_tools ......................   Passed    0.00 sec
      Start  8: coreUt_tuple3d
 8/43 Test  #8: coreUt_tuple3d ....................   Passed    0.00 sec
      Start  9: sysUt_devCpu
 9/43 Test  #9: sysUt_devCpu ......................   Passed    0.96 sec
      Start 10: sysUt_devGpu
10/43 Test #10: sysUt_devGpu ......................   Passed    1.09 sec
      Start 11: sysUt_devGpuNvcc
11/43 Test #11: sysUt_devGpuNvcc ..................   Passed    1.10 sec
      Start 12: sysUt_mem
12/43 Test #12: sysUt_mem .........................   Passed    1.14 sec
      Start 13: sysUt_mem3d
13/43 Test #13: sysUt_mem3d .......................   Passed    1.52 sec
      Start 14: sysUt_patterns
14/43 Test #14: sysUt_patterns ....................   Passed    2.17 sec
      Start 15: sysUt_report
15/43 Test #15: sysUt_report ......................   Passed    1.00 sec
      Start 16: setUt_gpuSet
16/43 Test #16: setUt_gpuSet ......................   Passed    2.18 sec
      Start 17: setUt_gpuSetNvcc
17/43 Test #17: setUt_gpuSetNvcc ..................   Passed    0.90 sec
      Start 18: setUt_memMirrorSet
18/43 Test #18: setUt_memMirrorSet ................   Passed    2.18 sec
      Start 19: setUt_patterns
19/43 Test #19: setUt_patterns ....................   Passed    1.86 sec
      Start 20: setUt_multiDeviceObject
20/43 Test #20: setUt_multiDeviceObject ...........   Passed    1.14 sec
      Start 21: setUt_containerGraph
21/43 Test #21: setUt_containerGraph ..............   Passed    5.38 sec
      Start 22: gUt_containers
22/43 Test #22: gUt_containers ....................   Passed    1.51 sec
      Start 23: gUt_containers
23/43 Test #23: gUt_containers ....................   Passed    1.63 sec
      Start 24: gUt_dataView_patterns
24/43 Test #24: gUt_dataView_patterns .............   Passed  357.79 sec
      Start 25: domain-unit-test-map
25/43 Test #25: domain-unit-test-map ..............   Passed   13.19 sec
      Start 26: domain-unit-test-cast
26/43 Test #26: domain-unit-test-cast .............   Passed   13.03 sec
      Start 27: gUt_patterns_container
27/43 Test #27: gUt_patterns_container ............   Passed  551.06 sec
      Start 28: domainUt_swap
28/43 Test #28: domainUt_swap .....................   Passed   62.10 sec
      Start 29: domain-unit-test-staggered-grid
29/43 Test #29: domain-unit-test-staggered-grid ...   Passed   37.65 sec
      Start 30: gUt_tools
30/43 Test #30: gUt_tools .........................   Passed    1.33 sec
      Start 31: gUt_vtk
31/43 Test #31: gUt_vtk ...........................   Passed    1.80 sec
      Start 32: gUt_bGrid
32/43 Test #32: gUt_bGrid .........................   Passed    1.29 sec
      Start 33: sUt_skeleton
33/43 Test #33: sUt_skeleton ......................   Passed  183.73 sec
      Start 34: sUt_skeletonOnStreams
34/43 Test #34: sUt_skeletonOnStreams .............   Passed  1245.07 sec
      Start 35: sUt_userInterface
35/43 Test #35: sUt_userInterface .................   Passed  141.43 sec
      Start 36: sUt_stencil
36/43 Test #36: sUt_stencil .......................   Passed    3.09 sec
      Start 37: solverUt_Poisson
37/43 Test #37: solverUt_Poisson ..................   Passed    0.89 sec
      Start 38: solverPt_Poisson
38/43 Test #38: solverPt_Poisson ..................   Passed    3.52 sec
      Start 39: tutorial-staggered-grid
39/43 Test #39: tutorial-staggered-grid ...........   Passed    1.28 sec
      Start 40: app-fractal
40/43 Test #40: app-fractal .......................   Passed    1.22 sec
      Start 41: app-lbm
41/43 Test #41: app-lbm ...........................   Passed   18.42 sec
      Start 42: app-gameOfLife
42/43 Test #42: app-gameOfLife ....................   Passed   12.18 sec
      Start 43: app-poisson
43/43 Test #43: app-poisson .......................   Passed    3.95 sec

100% tests passed, 0 tests failed out of 43

Total Test time (real) = 2725.35 sec

@massimim massimim requested review from Ahdhn and removed request for Ahdhn November 23, 2022 22:31
@massimim massimim requested a review from Ahdhn November 25, 2022 23:12
@massimim
Copy link
Collaborator Author

massimim commented Nov 25, 2022

An unit test (domain-unit-test-staggered-grid NodeToVoxels.dGrid) highlighted an issue on a staggered grid test that is not related to the skeleton. The effected unit test has been temporarily disabled.

@massimim
Copy link
Collaborator Author

bGrid unit tests in sUt_skeletonOnStreams have been temporary disabled until bGrid is extended to multi-GPUs.

@massimim massimim marked this pull request as ready for review November 25, 2022 23:20
Copy link
Collaborator

@Ahdhn Ahdhn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good! I just left a minor comment

@@ -53,7 +53,12 @@ LIBNEONCORE_EXPORT extern Logger LoggerObj;

} // namespace Neon

#if defined( NEON_ACTIVETE_TRACING)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we don't need this. If you want to turn the logging completely off you can just do Neon::globalSpace::LoggerObj.set_level(spdlog::level::off)

@Ahdhn
Copy link
Collaborator

Ahdhn commented Dec 1, 2022

There are a couple of failed tests on Windows. Here is the complete report
CTestNeonWindowsReport.log

@massimim
Copy link
Collaborator Author

massimim commented Dec 5, 2022

Part of the test that are failing are not related to the skeleton. What window system are you using?

@Ahdhn
Copy link
Collaborator

Ahdhn commented Dec 9, 2022

Windows logs

Create new tag: 20221209-0432 - Experimental
Cannot find file: E:/Github/Neon/temp/Neon/build/DartConfiguration.tcl
Test project E:/Github/Neon/temp/Neon/build
      Start  1: coreUt_chrono
 1/43 Test  #1: coreUt_chrono .....................   Passed   36.34 sec
      Start  2: coreUt_cli
 2/43 Test  #2: coreUt_cli ........................   Passed    0.18 sec
      Start  3: coreUt_digraph
 3/43 Test  #3: coreUt_digraph ....................   Passed    0.21 sec
      Start  4: coreUt_exceptions
 4/43 Test  #4: coreUt_exceptions .................   Passed    0.57 sec
      Start  5: coreUt_io
 5/43 Test  #5: coreUt_io .........................   Passed   34.94 sec
      Start  6: coreUt_logging
 6/43 Test  #6: coreUt_logging ....................   Passed    0.23 sec
      Start  7: coreUt_tools
 7/43 Test  #7: coreUt_tools ......................   Passed    0.16 sec
      Start  8: coreUt_tuple3d
 8/43 Test  #8: coreUt_tuple3d ....................   Passed    0.21 sec
      Start  9: sysUt_devCpu
 9/43 Test  #9: sysUt_devCpu ......................   Passed    0.38 sec
      Start 10: sysUt_devGpu
10/43 Test #10: sysUt_devGpu ......................   Passed    0.52 sec
      Start 11: sysUt_devGpuNvcc
11/43 Test #11: sysUt_devGpuNvcc ..................   Passed    0.48 sec
      Start 12: sysUt_mem
12/43 Test #12: sysUt_mem .........................   Passed   10.54 sec
      Start 13: sysUt_mem3d
13/43 Test #13: sysUt_mem3d .......................   Passed    0.75 sec
      Start 14: sysUt_patterns
14/43 Test #14: sysUt_patterns ....................   Passed    2.11 sec
      Start 15: sysUt_report
15/43 Test #15: sysUt_report ......................   Passed    0.29 sec
      Start 16: setUt_gpuSet
16/43 Test #16: setUt_gpuSet ......................   Passed    0.47 sec
      Start 17: setUt_gpuSetNvcc
17/43 Test #17: setUt_gpuSetNvcc ..................   Passed    0.25 sec
      Start 18: setUt_memMirrorSet
18/43 Test #18: setUt_memMirrorSet ................   Passed    0.49 sec
      Start 19: setUt_patterns
19/43 Test #19: setUt_patterns ....................   Passed    1.71 sec
      Start 20: setUt_multiDeviceObject
20/43 Test #20: setUt_multiDeviceObject ...........   Passed    0.52 sec
      Start 21: setUt_containerGraph
21/43 Test #21: setUt_containerGraph ..............   Passed    3.37 sec
      Start 22: gUt_containers
22/43 Test #22: gUt_containers ....................   Passed    0.56 sec
      Start 23: gUt_containers
23/43 Test #23: gUt_containers ....................   Passed    0.34 sec
      Start 24: gUt_dataView_patterns
24/43 Test #24: gUt_dataView_patterns .............   Passed  596.95 sec
      Start 25: domain-unit-test-map
25/43 Test #25: domain-unit-test-map ..............   Passed    3.83 sec
      Start 26: domain-unit-test-cast
26/43 Test #26: domain-unit-test-cast .............   Passed    3.81 sec
      Start 27: gUt_patterns_container
27/43 Test #27: gUt_patterns_container ............   Passed  866.23 sec
      Start 28: domainUt_swap
28/43 Test #28: domainUt_swap .....................   Passed   11.00 sec
      Start 29: domain-unit-test-staggered-grid
29/43 Test #29: domain-unit-test-staggered-grid ...   Passed    0.34 sec
      Start 30: gUt_tools
30/43 Test #30: gUt_tools .........................   Passed    0.37 sec
      Start 31: gUt_vtk
31/43 Test #31: gUt_vtk ...........................   Passed    0.62 sec
      Start 32: gUt_bGrid
32/43 Test #32: gUt_bGrid .........................   Passed    0.49 sec
      Start 33: sUt_skeleton
33/43 Test #33: sUt_skeleton ......................   Passed  132.95 sec
      Start 34: sUt_skeletonOnStreams
34/43 Test #34: sUt_skeletonOnStreams .............   Passed  674.49 sec
      Start 35: sUt_userInterface
35/43 Test #35: sUt_userInterface .................   Passed  186.60 sec
      Start 36: sUt_stencil
36/43 Test #36: sUt_stencil .......................   Passed    1.15 sec
      Start 37: solverUt_Poisson
37/43 Test #37: solverUt_Poisson ..................   Passed    0.64 sec
      Start 38: solverPt_Poisson
38/43 Test #38: solverPt_Poisson ..................   Passed    7.13 sec
      Start 39: tutorial-staggered-grid
39/43 Test #39: tutorial-staggered-grid ...........   Passed    0.38 sec
      Start 40: app-fractal
40/43 Test #40: app-fractal .......................   Passed    0.81 sec
      Start 41: app-lbm
41/43 Test #41: app-lbm ...........................   Passed   33.50 sec
      Start 42: app-gameOfLife
42/43 Test #42: app-gameOfLife ....................   Passed   21.41 sec
      Start 43: app-poisson
43/43 Test #43: app-poisson .......................   Passed    5.29 sec

100% tests passed, 0 tests failed out of 43

Total Test time (real) = 2643.83 sec

@massimim massimim merged commit 7a180a4 into develop Dec 9, 2022
@massimim massimim deleted the skeleton branch December 9, 2022 15:24
@github-actions github-actions bot locked and limited conversation to collaborators Dec 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants