Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Refactor igraph.Graph output of Nerve, modify
fit
behaviour of Nerv…
…e, add `store_edge_elements` kwarg to Nerve and make_mapper_pipeline, add Nerve and ParallelClustering to docs (#447) * Refactor igraph.Graph output, add Nerve and ParallelClustering to docs, add store_edge_elements kwarg to Nerve and make_mapper_pipeline - Store node metadata not as a graph-level dictionary, but as vertex attributes accessible by graph.vs[attr_name][node_id] or graph.vs[node_id][attr_name] for attr_name in ["pullback_set_label", "partial_cluster_label", "node_elements"] - Remove "node_id" from node attributes as it always coincided with the igraph.Graph node number anyway. - Automatically store sizes of intersections as edge weights, accessible by graph.es["weight"]. - Add "store_intersections" kwarg to Nerve and make_mapper_pipeline to allow storing indices of node intersections as edge attributes, accessible via graph.es["edge_elements"]. - Simplify logic of Nerve.fit_transform code - Change the attributes stored by Nerve.fit. Now the entire graph is stored as graph_ instead. - Improve documentation of make_mapper_pipeline - Expose ParallelClustering and Nerve in __init__ and docs. - Adapt tests, mapper quickstart notebook, and mapper plotting functions. * Add two tests for the behaviour of store_edge_elements and min_intersection * Remove check for shape of `layout` in _calculate_graph_data `layout` can only be a string or a callable, not an ndarray * Improve test coverage of mapper visualization modules * Create tests for plot_betti_surfaces and plot_betti_curves * Add plotly_params to remaining plot methods in diagrams/representations, missed out in #441 * Fix some linting and docstrings * Minor improvements * Avoid shadowing range function in plot_diagram
- Loading branch information