Skip to content

Commit dcfc066

Browse files
authored
CMake: prefix monitoring option with SUNDIALS_ (#867)
`SUNDIALS_BUILD_WITH_MONITORING` -> `SUNDIALS_ENABLE_MONITORING`
1 parent 1a82e26 commit dcfc066

File tree

15 files changed

+42
-25
lines changed

15 files changed

+42
-25
lines changed

CHANGELOG.md

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,7 @@ table below lists the old CMake option names and the new replacements.
6161
| `KOKKOS_KERNELS_WORKS` | `SUNDIALS_ENABLE_KOKKOS_KERNELS_CHECKS` |
6262
| `BUILD_FORTRAN_MODULE_INTERFACE` | `SUNDIALS_ENABLE_FORTRAN` |
6363
| `SUNDIALS_BUILD_WITH_PROFILING` | `SUNDIALS_ENABLE_PROFILING` |
64+
| `SUNDIALS_BUILD_WITH_MONITORING` | `SUNDIALS_ENABLE_MONITORING` |
6465
| `BUILD_BENCHMARKS` | `SUNDIALS_ENABLE_BENCHMARKS` |
6566
| `BENCHMARKS_INSTALL_PATH` | `SUNDIALS_BENCHMARKS_INSTALL_PATH` |
6667
| `SUNDIALS_BENCHMARK_OUTPUT_DIR` | `SUNDIALS_BENCHMARKS_OUTPUT_DIR` |
@@ -71,9 +72,10 @@ table below lists the old CMake option names and the new replacements.
7172
Following the updated CMake options, the macros listed below have been
7273
deprecated and replaced with versions that align with the new CMake options.
7374

74-
| Old Macro | New Macro |
75-
|---------------------------------|-----------------------------|
76-
| `SUNDIALS_BUILD_WITH_PROFILING` | `SUNDIALS_ENABLE_PROFILING` |
75+
| Old Macro | New Macro |
76+
|----------------------------------|------------------------------|
77+
| `SUNDIALS_BUILD_WITH_PROFILING` | `SUNDIALS_ENABLE_PROFILING` |
78+
| `SUNDIALS_BUILD_WITH_MONITORING` | `SUNDIALS_ENABLE_MONITORING` |
7779

7880
## Changes to SUNDIALS in release 7.6.0
7981

cmake/SundialsBuildOptionsPre.cmake

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,8 @@ set(SUNDIALS_COUNTER_TYPE
7171
# ---------------------------------------------------------------
7272

7373
set(DOCSTR "Build with simulation monitoring capabilities enabled")
74-
sundials_option(SUNDIALS_BUILD_WITH_MONITORING BOOL "${DOCSTR}" OFF)
74+
sundials_option(SUNDIALS_ENABLE_MONITORING BOOL "${DOCSTR}" OFF
75+
DEPRECATED_NAMES SUNDIALS_BUILD_WITH_MONITORING)
7576

7677
# ---------------------------------------------------------------
7778
# Option to enable profiling

doc/cvode/guide/source/Usage/index.rst

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -981,7 +981,7 @@ Main solver optional input functions
981981

982982
.. warning::
983983

984-
Modifying the solution in this function will result in undefined behavior. This function is only intended to be used for monitoring the integrator. SUNDIALS must be built with the CMake option ``SUNDIALS_BUILD_WITH_MONITORING``, to utilize this function. See :numref:`Installation` for more information.
984+
Modifying the solution in this function will result in undefined behavior. This function is only intended to be used for monitoring the integrator. SUNDIALS must be built with the CMake option ``SUNDIALS_ENABLE_MONITORING``, to utilize this function. See :numref:`Installation` for more information.
985985

986986
.. c:function:: int CVodeSetMonitorFrequency(void* cvode_mem, long int nst)
987987
@@ -1003,7 +1003,7 @@ Main solver optional input functions
10031003

10041004
.. warning::
10051005

1006-
Modifying the solution in this function will result in undefined behavior. This function is only intended to be used for monitoring the integrator. SUNDIALS must be built with the CMake option ``SUNDIALS_BUILD_WITH_MONITORING``, to utilize this function. See :numref:`Installation` for more information.
1006+
Modifying the solution in this function will result in undefined behavior. This function is only intended to be used for monitoring the integrator. SUNDIALS must be built with the CMake option ``SUNDIALS_ENABLE_MONITORING``, to utilize this function. See :numref:`Installation` for more information.
10071007

10081008
.. c:function:: int CVodeSetMaxOrd(void* cvode_mem, int maxord)
10091009

doc/cvodes/guide/source/Usage/SIM.rst

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -987,7 +987,7 @@ Main solver optional input functions
987987

988988
.. warning::
989989

990-
Modifying the solution in this function will result in undefined behavior. This function is only intended to be used for monitoring the integrator. SUNDIALS must be built with the CMake option ``SUNDIALS_BUILD_WITH_MONITORING``, to utilize this function. See :numref:`Installation` for more information.
990+
Modifying the solution in this function will result in undefined behavior. This function is only intended to be used for monitoring the integrator. SUNDIALS must be built with the CMake option ``SUNDIALS_ENABLE_MONITORING``, to utilize this function. See :numref:`Installation` for more information.
991991

992992
.. c:function:: int CVodeSetMonitorFrequency(void* cvode_mem, long int nst)
993993
@@ -1009,7 +1009,7 @@ Main solver optional input functions
10091009

10101010
.. warning::
10111011

1012-
Modifying the solution in this function will result in undefined behavior. This function is only intended to be used for monitoring the integrator. SUNDIALS must be built with the CMake option ``SUNDIALS_BUILD_WITH_MONITORING``, to utilize this function. See :numref:`Installation` for more information.
1012+
Modifying the solution in this function will result in undefined behavior. This function is only intended to be used for monitoring the integrator. SUNDIALS must be built with the CMake option ``SUNDIALS_ENABLE_MONITORING``, to utilize this function. See :numref:`Installation` for more information.
10131013

10141014
.. c:function:: int CVodeSetMaxOrd(void* cvode_mem, int maxord)
10151015

doc/shared/Changelog.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2402,7 +2402,7 @@ contact the SUNDIALS team about any performance changes that they notice.
24022402
Added new capabilities for monitoring the solve phase in the Newton and
24032403
fixed-point :c:type:`SUNNonlinearSolver`, and the SUNDIALS iterative linear
24042404
solvers. SUNDIALS must be built with the CMake option
2405-
:cmakeop:`SUNDIALS_BUILD_WITH_MONITORING` to use these capabilities.
2405+
``SUNDIALS_BUILD_WITH_MONITORING`` to use these capabilities.
24062406

24072407
Added specialized fused CUDA kernels to CVODE which may offer better performance
24082408
on smaller problems when using CVODE with the CUDA vector. See the optional

doc/shared/RecentChanges.rst

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -100,6 +100,8 @@ table below lists the old CMake option names and the new replacements.
100100
+-------------------------------------------+---------------------------------------------------------+
101101
| ``SUNDIALS_BUILD_WITH_PROFILING`` | :cmakeop:`SUNDIALS_ENABLE_PROFILING` |
102102
+-------------------------------------------+---------------------------------------------------------+
103+
| ``SUNDIALS_BUILD_WITH_MONITORING`` | :cmakeop:`SUNDIALS_ENABLE_MONITORING` |
104+
+-------------------------------------------+---------------------------------------------------------+
103105
| ``BUILD_BENCHMARKS`` | :cmakeop:`SUNDIALS_ENABLE_BENCHMARKS` |
104106
+-------------------------------------------+---------------------------------------------------------+
105107
| ``BENCHMARKS_INSTALL_PATH`` | :cmakeop:`SUNDIALS_BENCHMARKS_INSTALL_PATH` |
@@ -116,8 +118,10 @@ table below lists the old CMake option names and the new replacements.
116118
Following the updated CMake options, the macros listed below have been
117119
deprecated and replaced with versions that align with the new CMake options.
118120

119-
+-----------------------------------+-------------------------------+
120-
| Old Macro | New Macro |
121-
+-----------------------------------+-------------------------------+
122-
| ``SUNDIALS_BUILD_WITH_PROFILING`` | ``SUNDIALS_ENABLE_PROFILING`` |
123-
+-----------------------------------+-------------------------------+
121+
+------------------------------------+--------------------------------+
122+
| Old Macro | New Macro |
123+
+------------------------------------+--------------------------------+
124+
| ``SUNDIALS_BUILD_WITH_PROFILING`` | ``SUNDIALS_ENABLE_PROFILING`` |
125+
+------------------------------------+--------------------------------+
126+
| ``SUNDIALS_BUILD_WITH_MONITORING`` | ``SUNDIALS_ENABLE_MONITORING`` |
127+
+------------------------------------+--------------------------------+

doc/shared/sundials/Install.rst

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -800,7 +800,7 @@ For more information on logging in SUNDIALS, see :ref:`SUNDIALS.Logging`.
800800
Monitoring
801801
^^^^^^^^^^
802802

803-
.. cmakeoption:: SUNDIALS_BUILD_WITH_MONITORING
803+
.. cmakeoption:: SUNDIALS_ENABLE_MONITORING
804804

805805
Build SUNDIALS with capabilities for fine-grained monitoring of solver
806806
progress and statistics. This is primarily useful for debugging.
@@ -812,6 +812,10 @@ Monitoring
812812
Building with monitoring may result in minor performance degradation even
813813
if monitoring is not utilized.
814814

815+
.. versionadded:: x.y.z
816+
817+
Replaces the deprecated option ``SUNDIALS_BUILD_WITH_MONITORING``
818+
815819
.. _Installation.Options.Profiling:
816820

817821
Profiling

examples/arkode/C_serial/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ set(ARKODE_examples
101101
"ark_robertson.c\;\;exclude-single"
102102
"ark_twowaycouple_mri.c\;\;develop")
103103

104-
if(SUNDIALS_BUILD_WITH_MONITORING)
104+
if(SUNDIALS_ENABLE_MONITORING)
105105
list(APPEND ARKODE_examples "ark_brusselator_fp.c\;1\;exclude-single")
106106
endif()
107107

examples/cvode/serial/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ set(CVODE_examples
4141
"cvRoberts_dns_uw\;\;develop"
4242
"cvRocket_dns\;\;develop")
4343

44-
if(SUNDIALS_BUILD_WITH_MONITORING)
44+
if(SUNDIALS_ENABLE_MONITORING)
4545
list(APPEND CVODE_examples "cvKrylovDemo_ls\;0 1\;develop")
4646
endif()
4747

examples/cvodes/serial/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ set(CVODES_examples
5454
"cvsRoberts_dns_constraints\;\;develop"
5555
"cvsRoberts_dns_uw\;\;develop")
5656

57-
if(SUNDIALS_BUILD_WITH_MONITORING)
57+
if(SUNDIALS_ENABLE_MONITORING)
5858
list(APPEND CVODE_examples "cvsKrylovDemo_ls\;0 1\;develop")
5959
endif()
6060

0 commit comments

Comments
 (0)