Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

remove_edges() does not remove the specified edges #111

Closed
johannagehlen opened this issue May 8, 2023 · 1 comment
Closed

remove_edges() does not remove the specified edges #111

johannagehlen opened this issue May 8, 2023 · 1 comment

Comments

@johannagehlen
Copy link

I passed a non-empty list containing the edge numbers to remove_edges(), all of which are present in the hypergraph, but it does not remove any edges in the hypergraph.

This is my code:

list_to_remove = [1, 2, 3]
HG_removed = HG.remove_edges(list_to_remove)

but when I check:
print(len(HG.edges())
and
print(len(HG_removed.edges())

they are both still at 1000.

Thanks in advance for the help!
@brendapraggastis
Copy link
Collaborator

@johannagehlen We will be releasing HNX 2.0 on Saturday May 13. You will be able to remove edges using a list of edge ids. Add edge capabilities won't be supported in this release though.

bonicim added a commit that referenced this issue Oct 26, 2023
@bonicim
Pull request #111: HYP-177 update tests suite for entityset
7da6b3f 
Merge in HYP/hypernetx from HYP-177-update-tests-suite-for-entityset to develop

* commit 'a249417bb8efe6d14e91e18b617a4af460f77d70': (27 commits)
  HYP-177 Reorg entityset tests
  HYP-353 Add deprecation warnings for property column args
  HYP-353 Remove option to customize misc cell props col
  HYP-353 Remove option to customize misc props column
  HYP-356 Add deprecate warnings to certain ES methods
  HYP-177 Update tox.ini script test deps
  HYP-177 Return none when property not found; update tests
  HYP-177 Refactor and fix set_cell_property
  HYP-177 Cleanup tests
  HYP-177 Modify helper method
  HYP-177 Update pytest and tox config
  HYP-177 Reorganize tests; cleanup fixtures
  HYP-177 Add tests for elements_by_column
  HYP-177 Add tests for collapse_identical_elements
  HYP-177 Update test config for CI
  HYP-177 Add tests for level method
  HYP-177 Fix set_cell_property bug
  HYP-177 Minor cleanup on assign_properties
  Add tests for assign_cell_properties
  HYP-177 Add tests for assign_properties, update docs
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@brendapraggastis @johannagehlen