These changes should be the same as the previous signatures provided - though the modularity function had inconsistency between the signature and the pyfunction macro specification in previous versions. This inconsistency is now fixed, but I'm unsure if this is a breaking change or not. Functionally it should not behave any differently, but the documented specification will now appear -- correctly -- to be different than the lies it told before. (#48)

Author: daxpryce

packages/pyo3/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "graspologic_native"
-version = "1.2.2"
+version = "1.2.3"
 authors = ["[email protected]"]
 edition = "2018"
 license = "MIT"

packages/pyo3/src/lib.rs

@@ -92,7 +92,7 @@ impl HierarchicalCluster {
 /// :raises ParameterRangeError: One of the parameters provided did not meet the requirements in the documentation.
 /// :raises UnsafeInducementError: An internal algorithm error. Please report with reproduction steps.
 #[pyfunction]
-#[pyo3(signature=(edges, /, starting_communities=None, resolution=1.0, randomness=0.001, iterations=1, use_modularity=true, seed=None, trials=1))]
+#[pyo3(signature=(/, edges, starting_communities=None, resolution=1.0, randomness=0.001, iterations=1, use_modularity=true, seed=None, trials=1))]
 fn leiden(
     py: Python,
     edges: Vec<Edge>,
@@ -170,7 +170,7 @@ fn leiden(
 /// :raises ParameterRangeError: One of the parameters provided did not meet the requirements in the documentation.
 /// :raises UnsafeInducementError: An internal algorithm error. Please report with reproduction steps.
 #[pyfunction]
-#[pyo3(signature=(edges, /, starting_communities=None, resolution=1.0, randomness=0.001, iterations=1, use_modularity=true, max_cluster_size=1000, seed=None))]
+#[pyo3(signature=(/, edges, starting_communities=None, resolution=1.0, randomness=0.001, iterations=1, use_modularity=true, max_cluster_size=1000, seed=None))]
 fn hierarchical_leiden(
     py: Python,
     edges: Vec<Edge>,
@@ -211,7 +211,7 @@ fn hierarchical_leiden(
 /// :return: The modularity of the community partitioning provided for the network.
 /// :rtype: float
 #[pyfunction]
-#[pyo3(signature=(edges, communities, /, resolution=1.0))]
+#[pyo3(signature=(/, edges, communities, resolution=1.0))]
 fn modularity(
     py: Python,
     edges: Vec<Edge>,

packages/pyo3/tests/test_leiden.py

@@ -19,7 +19,7 @@ def _get_edges(path):
 class TestLeiden(unittest.TestCase):
     def test_leiden(self):
         edges = _get_edges(sbm_graph)
-        modularity, partitions = gcn.leiden(edges, seed=seed)
+        modularity, partitions = gcn.leiden(edges=edges, seed=seed)
 
     def test_reiterative_leiden(self):
         """