Skip to content

Conversation

BeaMarton13
Copy link
Contributor

I used Cody for the functions.

  • By submitting this pull request, I assign the copyright of my contribution to The igraph development team.
Copy link
Member

@szhorvat szhorvat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great to me @BeaMarton13, other than the small nitpicks I noted!

@ntamas, can you also have a look please?

Copy link
Member

@ntamas ntamas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, apart from what Szabolcs already noted.

@szhorvat szhorvat requested a review from Copilot June 22, 2025 12:47
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for fluid communities in the Python interface of igraph.

  • Implements the fluid communities algorithm in the C and Python layers.
  • Updates the igraph API (init.py) to expose the new feature.
  • Includes several tests in tests/test_decomposition.py validating the expected behavior and error conditions.

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
tests/test_decomposition.py Added test cases for the fluid communities algorithm.
src/igraph/community.py Introduced the Python wrapper for the fluid communities method.
src/igraph/init.py Exposed the new fluid communities API in igraph’s public interface.
src/_igraph/graphobject.c Added the C binding for fluid communities.
@szhorvat szhorvat marked this pull request as ready for review June 24, 2025 14:13
@szhorvat szhorvat merged commit bdad808 into igraph:develop Jun 24, 2025
22 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants