Skip to content

Conversation

@mstn
Copy link
Contributor

@mstn mstn commented Jan 8, 2026

  • Feature in Add lax::open_hypergraph function to delete an edge #13
  • Changed panic behavior in delete_nodes to be aligned with delete_edges. Rationale: better to fail hard than swallow errors. Technically, it is a breaking change but I don't think we should document it (edge case + nobody uses delete_nodes yet).

@mstn mstn changed the title Add lax::open_hypergraph function to delete an edge Add lax::open_hypergraph function to delete an edge Jan 8, 2026
@mstn mstn marked this pull request as ready for review January 9, 2026 06:05
@statusfailed
Copy link
Contributor

Agreed on panic behaviour, thanks for pointing this out!

FYI: for future tests you can use the new_edge method instead of creating the adjacency array manually, e.g.:

      let mut h = OpenHypergraph::empty();
      let x0 = h.new_node(node_label);
      let x1 = h.new_node(node_label);
      let x2 = h.new_node(node_label);

      state.new_edge(edge_label, ([x0, x1], [x2]));

@statusfailed statusfailed merged commit c2e4257 into hellas-ai:master Jan 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants