Skip to content

delete_vertices_idx() and induced_subgraph_map() have inconsistent names #2310

@szhorvat

Description

@szhorvat

igraph_delete_vertices_idx() and igraph_induced_subgraph_map() perform comparable operations: they subset the graph's vertices, and output a mapping and inverse mapping between the old and new vertex IDs. However, their names are confusingly inconsistent.

I propose renaming igraph_delete_vertices_idx() to igraph_delete_vertices_map() on 0.11 only.

Note that on develop, the behaviour of these functions has already been changed (now the mapping uses 0-based indexing). So let's not do the renaming/deprecation on master.

@vtraag and @ntamas, any objections?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions