Skip to content

Commit

Permalink
BUG: Mark Module_FastBilateral as advanced
Browse files Browse the repository at this point in the history
This was non-advanced due to its high module compliance rating. However,
there are no other remote modules the same, and it is odd to just
have FastBilateral as non-advanced. A revisit to making it non-advanced
could be made after re-evaluating all the remote modules' compliance
rating.
  • Loading branch information
thewtex committed May 16, 2024
1 parent 8e928a1 commit f1c390a
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions CMake/ITKModuleRemote.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -199,17 +199,16 @@ function(itk_fetch_module _name _description)

if(NOT DEFINED Module_${_name})
option(Module_${_name} "(Remote-${MODULE_COMPLIANCE_LEVEL}) ${_description}" OFF)
mark_as_advanced(Module_${_name})
else()
# If Module_${_name} is set manually, put its value in the CACHE
option(Module_${_name} "(Remote-${MODULE_COMPLIANCE_LEVEL}) ${_description}" ${Module_${_name}})
endif()

if(${MODULE_COMPLIANCE_LEVEL} GREATER_EQUAL ${ITK_MINIMUM_COMPLIANCE_LEVEL})
set(Module_${_name}_VALID ON)
mark_as_advanced(CLEAR Module_${_name})
else()
set(Module_${_name}_VALID OFF)
mark_as_advanced(FORCE Module_${_name})
endif()
# message(INFO " MODULE_VALID Module_${_name}:${Module_${_name}_VALID}:${MODULE_COMPLIANCE_LEVEL}>=${ITK_MINIMUM_COMPLIANCE_LEVEL}")

Expand Down

0 comments on commit f1c390a

Please sign in to comment.