Skip to content

Commit

Permalink
unified logging infrastructure (#1426)
Browse files Browse the repository at this point in the history
* start moved to unified logging lib

* new core logging lib

* gen and use config header

* gen config fix

* simplify, more error checking and more tests

* add close to test

* wire init of new logging, keep old marcos wired

* add test for logging

* add test srouce

* prevent use of bad comm if user forgot to pass one

* windows fix and typo fix

* add ascent_logging lib to mk config

* msg and log rank options and mpi log tests

* address comments and a few tweaks
  • Loading branch information
cyrush authored Dec 17, 2024
1 parent 01505c6 commit b054d6d
Show file tree
Hide file tree
Showing 38 changed files with 1,812 additions and 38 deletions.
4 changes: 3 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,11 @@ and this project aspires to adhere to [Semantic Versioning](https://semver.org/s
- [email protected]
- [email protected]
- [email protected]
- [email protected]
- [email protected]
- [email protected]

### Added
- Added a new unified logging infrastructure.
- Added support for unstructured topologies with mixed elements types (for example, hexs and tets).
- Added support for `pyramid` and `wedge` elements.
- Added `sphere`, `cylinder`, `box`, and `plane` options to the slice filter.
Expand Down
3 changes: 3 additions & 0 deletions src/config/ascent_config.mk.in
Original file line number Diff line number Diff line change
Expand Up @@ -302,6 +302,7 @@ ASCENT_LIB_FLAGS = $(ASCENT_DEVICE_OBJECT) \
-lascent \
-lascent_flow \
-lascent_lodepng \
-lascent_logging \
$(ASCENT_ROVER_LIB_FLAGS) $(DRAY_LIB_FLAGS) $(ASCENT_VTKH_LIB_FLAGS) $(ASCENT_VTKM_LIB_FLAGS) $(VTKH_LIB_FLAGS) $(ASCENT_CONDUIT_LIB_FLAGS) $(ASCENT_MFEM_LIB_FLAGS) $(ASCENT_RAJA_LIB_FLAGS) $(ASCENT_UMPIRE_LIB_FLAGS) $(ASCENT_CAMP_LIB_FLAGS) $(ASCENT_OCCA_LIB_FLAGS) $(ASCENT_GENTEN_LIB_FLAGS) $(ASCENT_PYTHON_LIBS) $(ASCENT_CALIPER_LIB_FLAGS) $(ASCENT_ADIAK_LIB_FLAGS) $(ASCENT_OPENMP_LINK_FLAGS) $(ASCENT_CUDA_LIB_FLAGS)

# if using Ascent with mpi and/or cuda with a non-GPU sim
Expand All @@ -310,6 +311,7 @@ ASCENT_MPI_LIB_FLAGS = $(ASCENT_DEVICE_OBJECT_MPI) \
-lascent_mpi \
-lascent_flow \
-lascent_lodepng \
-lascent_logging \
$(ASCENT_ROVER_MPI_LIB_FLAGS) $(DRAY_MPI_LIB_FLAGS) $(ASCENT_VTKH_MPI_LIB_FLAGS) $(ASCENT_VTKM_LIB_FLAGS) $(VTKH_MPI_LIB_FLAGS) $(ASCENT_CONDUIT_MPI_LIB_FLAGS) $(ASCENT_MFEM_LIB_FLAGS) $(ASCENT_FIDES_LIB_FLAGS) $(ASCENT_ADIOS2_LIB_FLAGS) $(ASCENT_PMT_LIB_FLAGS) $(ASCENT_STREAMSTAT_LIB_FLAGS) $(ASCENT_TOPOFILEPARSER_LIB_FLAGS) $(ASCENT_BABELFLOW_LIB_FLAGS) $(ASCENT_RAJA_LIB_FLAGS) $(ASCENT_UMPIRE_LIB_FLAGS) $(ASCENT_CAMP_LIB_FLAGS) $(ASCENT_OCCA_LIB_FLAGS) $(ASCENT_GENTEN_LIB_FLAGS) $(ASCENT_PYTHON_LIBS) $(ASCENT_CALIPER_LIB_FLAGS) $(ASCENT_OPENMP_LINK_FLAGS) $(ASCENT_CUDA_LIB_FLAGS)


Expand All @@ -319,5 +321,6 @@ ASCENT_MPI_CUDA_LIB_FLAGS = -L $(ASCENT_DIR)/lib \
-lascent_mpi \
-lascent_flow \
-lascent_lodepng \
-lascent_logging \
$(ASCENT_ROVER_MPI_LIB_FLAGS) $(DRAY_MPI_LIB_FLAGS) $(ASCENT_VTKH_MPI_LIB_FLAGS) $(ASCENT_VTKM_LIB_FLAGS) $(VTKH_MPI_LIB_FLAGS) $(ASCENT_CONDUIT_MPI_LIB_FLAGS) $(ASCENT_MFEM_LIB_FLAGS) $(ASCENT_OCCA_LIB_FLAGS) $(ASCENT_GENTEN_LIB_FLAGS) $(ASCENT_FIDES_LIB_FLAGS) $(ASCENT_ADIOS2_LIB_FLAGS) $(ASCENT_PMT_LIB_FLAGS) $(ASCENT_STREAMSTAT_LIB_FLAGS) $(ASCENT_TOPOFILEPARSER_LIB_FLAGS) $(ASCENT_BABELFLOW_LIB_FLAGS) $(ASCENT_RAJA_LIB_FLAGS) $(ASCENT_UMPIRE_LIB_FLAGS) $(ASCENT_CAMP_LIB_FLAGS) $(ASCENT_PYTHON_LIBS) $(ASCENT_CALIPER_LIB_FLAGS) $(ASCENT_OPENMP_LINK_FLAGS) $(ASCENT_CUDA_LIB_FLAGS)

22 changes: 15 additions & 7 deletions src/libs/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2,14 +2,9 @@
# Project developers. See top-level LICENSE AND COPYRIGHT files for dates and
# other details. No copyright assignment is required to contribute to Ascent.


################################
# Add flow
################################
add_subdirectory(flow)


#######################################
# display info about optional features
#######################################
if(ENABLE_APCOMP)
message(STATUS "Building APComp (ENABLE_APCOMP=ON)")
else()
Expand All @@ -29,10 +24,23 @@ else()
endif()


################################
# ascent_png_utils object lib
# shared for testing of all libs
################################
add_subdirectory(png_utils)

################################
# ascent_logging lib
# shared logging for all libs
################################
add_subdirectory(logging)

################################
# Add flow
################################
add_subdirectory(flow)

################################
# Add ap comp
################################
Expand Down
9 changes: 4 additions & 5 deletions src/libs/ascent/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -171,15 +171,14 @@ set(ascent_headers
# utils
utils/ascent_actions_utils.hpp
utils/ascent_data_logger.hpp
utils/ascent_logging.hpp
utils/ascent_logging_old.hpp
utils/ascent_block_timer.hpp
utils/ascent_mpi_utils.hpp
utils/ascent_string_utils.hpp
utils/ascent_web_interface.hpp
utils/ascent_resources.hpp
utils/ascent_resources_cinema_web.hpp
utils/ascent_resources_ascent_web.hpp
utils/ascent_annotations.hpp
# hola
hola/ascent_hola.hpp)

Expand Down Expand Up @@ -238,12 +237,11 @@ set(ascent_sources
utils/ascent_actions_utils.cpp
utils/ascent_data_logger.cpp
utils/ascent_block_timer.cpp
utils/ascent_logging.cpp
utils/ascent_logging_old.cpp
utils/ascent_mpi_utils.cpp
utils/ascent_string_utils.cpp
utils/ascent_web_interface.cpp
utils/ascent_resources.cpp
utils/ascent_annotations.cpp
# hola
hola/ascent_hola.cpp)

Expand Down Expand Up @@ -371,7 +369,8 @@ set(ascent_thirdparty_libs
conduit::conduit
ascent_flow
ascent_lodepng
ascent_png_utils)
ascent_png_utils
ascent_logging)

##################
# optional libs
Expand Down
Loading

0 comments on commit b054d6d

Please sign in to comment.