fix(visibility): restrict RAFT_EXPORT to core/ and raft_runtime/ only

cpp/include/raft/common/detail/scatter.cuh

@@ -9,7 +9,7 @@
 #include <raft/util/cuda_utils.cuh>
 #include <raft/util/vectorized.cuh>
 
-namespace RAFT_EXPORT raft {
+namespace raft {
 namespace detail {
 
 template <typename DataT, int VecLen, typename Lambda, typename IdxT>
@@ -41,4 +41,4 @@ void scatterImpl(
 }
 
 }  // namespace detail
-}  // namespace RAFT_EXPORT raft
+}  // namespace raft

cpp/include/raft/comms/comms_test.hpp

@@ -10,7 +10,7 @@
 #include <raft/core/detail/macros.hpp>
 #include <raft/core/resources.hpp>
 
-namespace RAFT_EXPORT raft {
+namespace raft {
 namespace comms {
 
 /**
@@ -157,4 +157,4 @@ bool test_commsplit(raft::resources const& h, int n_colors)
   return detail::test_commsplit(h, n_colors);
 }
 }  // namespace comms
-};  // namespace RAFT_EXPORT raft
+};  // namespace raft

cpp/include/raft/comms/detail/mpi_comms.hpp

@@ -60,7 +60,7 @@
 #ifndef MPI_TRY_NO_THROW
 #define MPI_TRY_NO_THROW(call) RAFT_MPI_TRY_NO_THROW(call)
 #endif
-namespace RAFT_EXPORT raft {
+namespace raft {
 namespace comms {
 namespace detail {
 
@@ -457,4 +457,4 @@ class mpi_comms : public comms_iface {
 
 }  // end namespace detail
 };  // end namespace comms
-};  // namespace RAFT_EXPORT raft
+};  // namespace raft

cpp/include/raft/comms/detail/std_comms.hpp

@@ -36,7 +36,7 @@
 #include <unordered_set>
 #include <utility>
 
-namespace RAFT_EXPORT raft {
+namespace raft {
 namespace comms {
 namespace detail {
 
@@ -597,4 +597,4 @@ class std_comms : public comms_iface {
 };
 }  // namespace detail
 }  // end namespace comms
-}  // namespace RAFT_EXPORT raft
+}  // namespace raft

cpp/include/raft/comms/detail/test.hpp

@@ -17,7 +17,7 @@
 #include <iostream>
 #include <numeric>
 
-namespace RAFT_EXPORT raft {
+namespace raft {
 namespace comms {
 namespace detail {
 
@@ -531,4 +531,4 @@ bool test_commsplit(raft::resources const& h, int n_colors)
 
 }  // namespace detail
 }  // namespace comms
-};  // namespace RAFT_EXPORT raft
+};  // namespace raft

cpp/include/raft/comms/detail/ucp_helper.hpp

@@ -12,7 +12,7 @@
 #include <ucp/api/ucp.h>
 #include <ucp/api/ucp_def.h>
 
-namespace RAFT_EXPORT raft {
+namespace raft {
 namespace comms {
 namespace detail {
 
@@ -151,4 +151,4 @@ class comms_ucp_handler {
 };
 }  // end namespace detail
 }  // end namespace comms
-}  // namespace RAFT_EXPORT raft
+}  // namespace raft

cpp/include/raft/comms/detail/util.hpp

@@ -15,7 +15,7 @@
 
 #include <string>
 
-namespace RAFT_EXPORT raft {
+namespace raft {
 namespace comms {
 namespace detail {
 
@@ -97,4 +97,4 @@ inline status_t nccl_sync_stream(ncclComm_t comm, cudaStream_t stream)
 
 };  // namespace detail
 };  // namespace comms
-};  // namespace RAFT_EXPORT raft
+};  // namespace raft

cpp/include/raft/comms/mpi_comms.hpp

@@ -11,7 +11,7 @@
 #include <raft/core/resource/comms.hpp>
 #include <raft/core/resource/cuda_stream.hpp>
 
-namespace RAFT_EXPORT raft {
+namespace raft {
 namespace comms {
 
 using mpi_comms = detail::mpi_comms;
@@ -59,4 +59,4 @@ inline void initialize_mpi_comms(resources* handle, MPI_Comm comm)
  */
 
 };  // namespace comms
-};  // namespace RAFT_EXPORT raft
+};  // namespace raft

cpp/include/raft/comms/std_comms.hpp

@@ -18,7 +18,7 @@
 
 #include <iostream>
 
-namespace RAFT_EXPORT raft {
+namespace raft {
 namespace comms {
 
 using std_comms = detail::std_comms;
@@ -174,4 +174,4 @@ inline void get_nccl_unique_id(char* uid)
   memcpy(uid, id.internal, NCCL_UNIQUE_ID_BYTES);
 }
 };  // namespace comms
-};  // namespace RAFT_EXPORT raft
+};  // namespace raft

cpp/include/raft/core/cublas_macros.hpp

@@ -37,6 +37,9 @@ struct cublas_error : public raft::exception {
  * @}
  */
 
+}  // namespace RAFT_EXPORT raft
+
+namespace raft {
 namespace linalg {
 namespace detail {
 
@@ -59,7 +62,7 @@ inline const char* cublas_error_to_string(cublasStatus_t err)
 
 }  // namespace detail
 }  // namespace linalg
-}  // namespace RAFT_EXPORT raft
+}  // namespace raft
 
 #undef _CUBLAS_ERR_TO_STR
 

cpp/include/raft/core/cusolver_macros.hpp

@@ -38,6 +38,9 @@ struct cusolver_error : public raft::exception {
  * @}
  */
 
+}  // namespace RAFT_EXPORT raft
+
+namespace raft {
 namespace linalg {
 namespace detail {
 
@@ -60,7 +63,7 @@ inline const char* cusolver_error_to_string(cusolverStatus_t err)
 
 }  // namespace detail
 }  // namespace linalg
-}  // namespace RAFT_EXPORT raft
+}  // namespace raft
 
 #undef _CUSOLVER_ERR_TO_STR
 

cpp/include/raft/core/cusparse_macros.hpp

@@ -42,6 +42,9 @@ struct cusparse_error : public raft::exception {
 /**
  * @}
  */
+}  // namespace RAFT_EXPORT raft
+
+namespace raft {
 namespace sparse {
 namespace detail {
 
@@ -52,7 +55,7 @@ inline const char* cusparse_error_to_string(cusparseStatus_t err)
 
 }  // namespace detail
 }  // namespace sparse
-}  // namespace RAFT_EXPORT raft
+}  // namespace raft
 
 #undef _CUSPARSE_ERR_TO_STR
 

cpp/include/raft/core/detail/copy.hpp

@@ -27,7 +27,7 @@
 #endif
 #endif
 
-namespace RAFT_EXPORT raft {
+namespace raft {
 namespace detail {
 
 template <bool B,
@@ -543,4 +543,4 @@ mdspan_copyable_t<DstType, SrcType> copy(resources const& res, DstType&& dst, Sr
   }
 }
 }  // namespace detail
-}  // namespace RAFT_EXPORT raft
+}  // namespace raft

cpp/include/raft/core/detail/fail_container_policy.hpp

@@ -13,7 +13,7 @@
 
 #include <stddef.h>
 
-namespace RAFT_EXPORT raft {
+namespace raft {
 namespace detail {
 
 template <typename T>
@@ -135,4 +135,4 @@ struct fail_container_policy {
 };
 
 }  // namespace detail
-}  // namespace RAFT_EXPORT raft
+}  // namespace raft

cpp/include/raft/core/detail/mdspan_numpy_serializer.hpp

@@ -26,7 +26,7 @@
 #include <type_traits>
 #include <vector>
 
-namespace RAFT_EXPORT raft {
+namespace raft {
 
 namespace detail {
 
@@ -487,4 +487,4 @@ inline T deserialize_scalar(std::istream& is)
 
 }  // end namespace numpy_serializer
 }  // end namespace detail
-}  // namespace RAFT_EXPORT raft
+}  // namespace raft

cpp/include/raft/core/detail/mdspan_util.cuh

@@ -10,7 +10,7 @@
 #include <tuple>
 #include <utility>
 
-namespace RAFT_EXPORT raft {
+namespace raft {
 namespace detail {
 
 template <class T, std::size_t N, std::size_t... Idx>
@@ -58,4 +58,4 @@ RAFT_INLINE_FUNCTION auto popc(uint64_t v) -> int32_t
 }
 
 }  // namespace detail
-}  // namespace RAFT_EXPORT raft
+}  // namespace raft

cpp/include/raft/core/detail/nvtx.hpp

@@ -21,7 +21,7 @@
 #include <type_traits>
 #include <vector>
 
-namespace RAFT_EXPORT raft {
+namespace raft {
 namespace common::nvtx::detail {
 
 /**
@@ -178,11 +178,11 @@ inline void pop_range()
 }
 
 }  // namespace common::nvtx::detail
-}  // namespace RAFT_EXPORT raft
+}  // namespace raft
 
 #else   // NVTX_ENABLED
 
-namespace RAFT_EXPORT raft {
+namespace raft {
 namespace common::nvtx::detail {
 
 template <typename Domain, typename... Args>
@@ -196,5 +196,5 @@ inline void pop_range()
 }
 
 }  // namespace common::nvtx::detail
-}  // namespace RAFT_EXPORT raft
+}  // namespace raft
 #endif  // NVTX_ENABLED

cpp/include/raft/core/detail/nvtx_range_stack.hpp

@@ -13,7 +13,7 @@
 #include <string>
 #include <utility>
 
-namespace RAFT_EXPORT raft {
+namespace raft {
 namespace common::nvtx {
 
 namespace detail {
@@ -91,4 +91,4 @@ inline auto thread_local_current_range() -> std::shared_ptr<const current_range>
 }
 
 }  // namespace common::nvtx
-}  // namespace RAFT_EXPORT raft
+}  // namespace raft

cpp/include/raft/core/detail/span.hpp

@@ -10,7 +10,7 @@
 #include <limits>  // numeric_limits
 #include <type_traits>
 
-namespace RAFT_EXPORT raft {
+namespace raft {
 
 template <class ElementType, bool is_device, std::size_t Extent>
 class span;
@@ -95,4 +95,4 @@ struct span_storage<T, dynamic_extent> {
   [[nodiscard]] constexpr auto data() const noexcept -> T* { return ptr_; }
 };
 }  // namespace detail
-}  // namespace RAFT_EXPORT raft
+}  // namespace raft

cpp/include/raft/core/mdarray.hpp

@@ -50,6 +50,9 @@ class array_interface {
   auto view() const noexcept { return static_cast<Base*>(this)->view(); }
 };
 
+}  // namespace RAFT_EXPORT raft
+
+namespace raft {
 namespace detail {
 template <typename T, typename = void>
 struct is_array_interface : std::false_type {};
@@ -68,7 +71,9 @@ using is_array_interface_t = is_array_interface<std::remove_const_t<T>>;
 template <typename T>
 inline constexpr bool is_array_interface_v = is_array_interface<std::remove_const_t<T>>::value;
 }  // namespace detail
+}  // namespace raft
 
+namespace RAFT_EXPORT raft {
 template <typename...>
 struct is_array_interface : std::true_type {};
 template <typename T1>

cpp/include/raft/core/mdbuffer.cuh

@@ -69,6 +69,9 @@ using alternate_from_mem_type =
   std::variant_alternative_t<variant_index_from_memory_type(MemType) % std::variant_size_v<Variant>,
                              Variant>;
 
+}  // namespace RAFT_EXPORT raft
+
+namespace raft {
 namespace detail {
 template <typename T, raft::memory_type MemType>
 struct memory_type_to_default_policy {};
@@ -92,7 +95,9 @@ struct memory_type_to_default_policy<T, raft::memory_type::pinned> {
 template <typename T, raft::memory_type MemType>
 using memory_type_to_default_policy_t = typename memory_type_to_default_policy<T, MemType>::type;
 }  // namespace detail
+}  // namespace raft
 
+namespace RAFT_EXPORT raft {
 /**
  * @brief A variant of container policies for each memory type which can be
  * used to build the default container policy for a buffer.

cpp/include/raft/core/mdspan.hpp

@@ -25,6 +25,9 @@ template <typename ElementType,
           typename AccessorPolicy = cuda::std::default_accessor<ElementType>>
 using mdspan = cuda::std::mdspan<ElementType, Extents, LayoutPolicy, AccessorPolicy>;
 
+}  // namespace RAFT_EXPORT raft
+
+namespace raft {
 namespace detail {
 
 template <typename IntType>
@@ -53,7 +56,9 @@ struct alignment {
 };
 
 }  // namespace detail
+}  // namespace raft
 
+namespace RAFT_EXPORT raft {
 /**
  * @brief Padded layouts that take a padding value directly.
  */

cpp/include/raft/core/memory_type.hpp

@@ -56,6 +56,9 @@ struct memory_type_constant {
   }();
 };
 
+}  // namespace RAFT_EXPORT raft
+
+namespace raft {
 namespace detail {
 
 template <bool is_host_accessible, bool is_device_accessible>
@@ -73,7 +76,9 @@ auto constexpr memory_type_from_access()
 }
 
 }  // end namespace detail
+}  // namespace raft
 
+namespace RAFT_EXPORT raft {
 template <typename T>
 auto memory_type_from_pointer(T* ptr)
 {