Remove Deprecated API

cpp/include/raft/neighbors/brute_force_types.hpp

@@ -85,71 +85,6 @@ struct index : ann::index {
   auto operator=(index&&) -> index&      = default;
   ~index()                               = default;
 
-  /** Construct a brute force index from dataset
-   *
-   * Constructs a brute force index from a dataset. This lets us precompute norms for
-   * the dataset, providing a speed benefit over doing this at query time.
-
-   * If the dataset is already in GPU memory, then this class stores a non-owning reference to
-   * the dataset. If the dataset is in host memory, it will be copied to the device and the
-   * index will own the device memory.
-   */
-
-  template <typename data_accessor>
-  [[deprecated("Use cuVS instead")]] index(
-    raft::resources const& res,
-    mdspan<const T, matrix_extent<int64_t>, row_major, data_accessor> dataset,
-    std::optional<raft::device_vector<T, int64_t>>&& norms,
-    raft::distance::DistanceType metric,
-    T metric_arg = 0.0)
-    : ann::index(),
-      metric_(metric),
-      dataset_(make_device_matrix<T, int64_t>(res, 0, 0)),
-      norms_(std::move(norms)),
-      metric_arg_(metric_arg)
-  {
-    if (norms_) { norms_view_ = make_const_mdspan(norms_.value().view()); }
-    update_dataset(res, dataset);
-    resource::sync_stream(res);
-  }
-
-  /** Construct a brute force index from dataset
-   *
-   * This class stores a non-owning reference to the dataset and norms here.
-   * Having precomputed norms gives us a performance advantage at query time.
-   */
-  [[deprecated("Use cuVS instead")]] index(
-    raft::resources const& res,
-    raft::device_matrix_view<const T, int64_t, row_major> dataset_view,
-    std::optional<raft::device_vector_view<const T, int64_t>> norms_view,
-    raft::distance::DistanceType metric,
-    T metric_arg = 0.0)
-    : ann::index(),
-      metric_(metric),
-      dataset_(make_device_matrix<T, int64_t>(res, 0, 0)),
-      dataset_view_(dataset_view),
-      norms_view_(norms_view),
-      metric_arg_(metric_arg)
-  {
-  }
-
-  template <typename data_accessor>
-  [[deprecated("Use cuVS instead")]] index(
-    raft::resources const& res,
-    index_params const& params,
-    mdspan<const T, matrix_extent<int64_t>, row_major, data_accessor> dataset,
-    std::optional<raft::device_vector<T, int64_t>>&& norms = std::nullopt)
-    : ann::index(),
-      metric_(params.metric),
-      dataset_(make_device_matrix<T, int64_t>(res, 0, 0)),
-      norms_(std::move(norms)),
-      metric_arg_(params.metric_arg)
-  {
-    if (norms_) { norms_view_ = make_const_mdspan(norms_.value().view()); }
-    update_dataset(res, dataset);
-    resource::sync_stream(res);
-  }
-
   /**
    * Replace the dataset with a new dataset.
    */

cpp/include/raft/neighbors/cagra.cuh

@@ -385,12 +385,3 @@ void search(raft::resources const& res,
 /** @} */  // end group cagra
 
 }  // namespace raft::neighbors::cagra
-
-// TODO: Remove deprecated experimental namespace in 23.12 release
-namespace raft::neighbors::experimental::cagra {
-using raft::neighbors::cagra::build;
-using raft::neighbors::cagra::build_knn_graph;
-using raft::neighbors::cagra::optimize;
-using raft::neighbors::cagra::search;
-using raft::neighbors::cagra::sort_knn_graph;
-}  // namespace raft::neighbors::experimental::cagra

cpp/include/raft/neighbors/cagra_serialize.cuh

@@ -228,10 +228,3 @@ index<T, IdxT> deserialize(raft::resources const& handle, const std::string& fil
 /**@}*/
 
 }  // namespace raft::neighbors::cagra
-
-// TODO: Remove deprecated experimental namespace in 23.12 release
-namespace raft::neighbors::experimental::cagra {
-using raft::neighbors::cagra::deserialize;
-using raft::neighbors::cagra::serialize;
-
-}  // namespace raft::neighbors::experimental::cagra