Skip to content

relationship_ids filter in Mapping API - #29

Merged
alex-omophub merged 2 commits into
mainfrom
develop
Aug 11, 2026
Merged

relationship_ids filter in Mapping API #29
alex-omophub merged 2 commits into
mainfrom
develop

Conversation

@alex-omophub

@alex-omophub alex-omophub commented Aug 11, 2026

Copy link
Copy Markdown
Member

Summary by cubic

Add relationship_ids filter to Mappings.get() / get_iter() (sync and async) to control which relationship types are returned. This enables fetching composite concept decompositions by including "Maps to value" in addition to the server default "Maps to".

  • New Features
    • Accepts relationship_ids as a list or comma-separated string; sent as the relationship_ids query param.
    • Supported in get() and get_iter() and their async counterparts; forwarded through pagination.
    • Updated docs and CHANGELOG; unit tests cover default, list, and string cases.

Written for commit 6fff39f. Summary will update on new commits.

Review in cubic

alex-omophub added 2 commits August 11, 2026 23:05
…entation

- Introduced `relationship_ids` parameter in `Mappings.get()` and `AsyncMappings.get()` methods, allowing users to specify relationship types as a list or comma-separated string.
- Updated documentation to clarify the behavior of the new parameter, including examples for composite concepts.
- Modified the CHANGELOG to reflect these changes and their implications for API usage.
- Added unit tests to ensure correct handling of the `relationship_ids` parameter in various scenarios.
@alex-omophub alex-omophub changed the title Develop relationship_ids filter in Mapping API Aug 11, 2026
@codecov

codecov Bot commented Aug 11, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 50.00000% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/omophub/resources/mappings.py 50.00% 2 Missing and 2 partials ⚠️

📢 Thoughts on this report? Let us know!

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

Re-trigger cubic

@alex-omophub
alex-omophub merged commit 2919eeb into main Aug 11, 2026
13 of 14 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.

1 participant