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.

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

Docs/docstrings #169

Closed
wants to merge 8 commits into from
Closed

Docs/docstrings #169

wants to merge 8 commits into from

Conversation

ckmah
Copy link
Collaborator

@ckmah ckmah commented Dec 17, 2024

This pull request includes several updates to improve the clarity and functionality of the SpatialData processing functions. The main changes involve refining the function docstrings, adding error handling, and removing an unused function.

Docstring improvements and error handling:

  • bento/_utils.py: Updated docstrings for functions such as filter_by_gene, get_points, get_shape, get_points_metadata, get_shape_metadata, set_points_metadata, set_shape_metadata, _sync_points, _sync_shapes, get_instance_key, and get_feature_key to improve clarity and consistency. Added error handling for missing keys.

Codebase simplification:

  • bento/geometry/__init__.py: Removed the import of the unused labels_to_shapes function.
  • bento/geometry/_ops.py: Removed the labels_to_shapes function to simplify the codebase. Updated the overlay function docstring for clarity.

Functionality enhancements:

  • bento/io/_index.py: Improved the _sjoin_points and _sjoin_shapes functions by refining docstrings and adding error handling for missing keys. Added a check to ensure empty categories are present in shape keys.

General improvements:

  • bento/io/_io.py: Enhanced the prep function docstring for better readability and understanding.

@ckmah ckmah closed this Dec 17, 2024
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.

1 participant