Skip to content

Conversation

@prsai0110
Copy link

This pull request resolves issue #13226 by ensuring consistent behavior when handling missing values in coordinate_compression.py.

Changes made:

  • compress(value) now raises a ValueError if the value is not found in the coordinate map.
  • decompress(index) raises a ValueError for out-of-bounds indices.
  • Added docstrings and doctests to clarify expected behavior.
  • Verified changes using pytest and doctest.

Pull Request Checklist:

  • I have read CONTRIBUTING.md.
  • I have followed the formatting guidelines in the code style guide.
  • I have added relevant tests.
  • I have added documentation to all new and modified functions.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have used Python's typing annotations.

@algorithms-keeper algorithms-keeper bot added the tests are failing Do not merge until tests pass label Nov 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests are failing Do not merge until tests pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant