Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove legacy leiden #3581

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion cpp/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -213,7 +213,6 @@ set(CUGRAPH_SOURCES
src/community/leiden_sg.cu
src/community/leiden_mg.cu
src/community/legacy/louvain.cu
src/community/legacy/leiden.cu
src/community/legacy/ktruss.cu
src/community/legacy/ecg.cu
src/community/egonet_sg.cu
Expand Down
41 changes: 0 additions & 41 deletions cpp/include/cugraph/algorithms.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -677,47 +677,6 @@ void flatten_dendrogram(raft::handle_t const& handle,
Dendrogram<typename graph_view_t::vertex_type> const& dendrogram,
typename graph_view_t::vertex_type* clustering);

/**
* @brief Legacy Leiden implementation
*
* Compute a clustering of the graph by maximizing modularity using the Leiden improvements
* to the Louvain method.
*
* Computed using the Leiden method described in:
*
* Traag, V. A., Waltman, L., & van Eck, N. J. (2019). From Louvain to Leiden:
* guaranteeing well-connected communities. Scientific reports, 9(1), 5233.
* doi: 10.1038/s41598-019-41695-z
*
* @throws cugraph::logic_error when an error occurs.
*
* @tparam vertex_t Type of vertex identifiers.
* Supported value : int (signed, 32-bit)
* @tparam edge_t Type of edge identifiers.
* Supported value : int (signed, 32-bit)
* @tparam weight_t Type of edge weights. Supported values : float or double.
*
* @param[in] handle Library handle (RAFT). If a communicator is set in the handle,
* @param[in] graph input graph object (CSR)
* @param[out] clustering Pointer to device array where the clustering should be stored
* @param[in] max_level (optional) maximum number of levels to run (default 100)
* @param[in] resolution (optional) The value of the resolution parameter to use.
* Called gamma in the modularity formula, this changes the size
* of the communities. Higher resolutions lead to more smaller
* communities, lower resolutions lead to fewer larger
* communities. (default 1)
*
* @return a pair containing:
* 1) number of levels of the returned clustering
* 2) modularity of the returned clustering
*/
template <typename vertex_t, typename edge_t, typename weight_t>
std::pair<size_t, weight_t> leiden(raft::handle_t const& handle,
legacy::GraphCSRView<vertex_t, edge_t, weight_t> const& graph,
vertex_t* clustering,
size_t max_level = 100,
weight_t resolution = weight_t{1});

/**
* @brief Leiden implementation
*
Expand Down
74 changes: 0 additions & 74 deletions cpp/src/community/legacy/leiden.cu

This file was deleted.

173 changes: 0 additions & 173 deletions cpp/src/community/legacy/leiden.cuh

This file was deleted.

1 change: 0 additions & 1 deletion cpp/tests/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -217,7 +217,6 @@ ConfigureTest(LOUVAIN_TEST community/louvain_test.cpp)

###################################################################################################
# - LEIDEN tests ---------------------------------------------------------------------------------
ConfigureTest(LEIDEN_TEST community/leiden_test.cpp)
ConfigureTest(NEW_LEIDEN_TEST community/new_leiden_test.cpp)

###################################################################################################
Expand Down
85 changes: 0 additions & 85 deletions cpp/tests/community/leiden_test.cpp

This file was deleted.