Conversation
…ructured dataframe
… from tabular data and YAML config
There was a problem hiding this comment.
Pull Request Overview
This pull request introduces a new mapping lookup table (LUT) feature for ontology term translation, enabling easier mapping and translation of ontology terms across different databases. The implementation includes comprehensive testing, utility functions, and API enhancements to support the new functionality.
- Adds
MappingLUTclass and utilities for reading mapping files and creating reverse mapping dictionaries - Enhances
ClientOntologywith methods for loading mapping files, retrieving LUTs, and batch term retrieval - Updates pre-commit hook versions and adds pandas dependency
Reviewed Changes
Copilot reviewed 9 out of 10 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| ontograph/utils.py | Core utilities for reading mapping files, creating reverse mappings, and translating term IDs |
| ontograph/models.py | New MappingLUT class to represent mapping lookup tables |
| ontograph/client.py | Client API enhancements for mapping operations and batch term retrieval |
| ontograph/queries/navigator.py | Updated navigator to support batch term retrieval with improved error handling |
| tests/test_utils.py | Comprehensive test suite for utility functions |
| tests/test_models.py | Test coverage for the new MappingLUT class |
| pyproject.toml | Adds pandas dependency |
| notebooks/chebi_mix_ontology.ipynb | Demonstration notebook showing mapping LUT workflow |
| .pre-commit-config.yaml | Updates pre-commit hook versions |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 9 out of 10 changed files in this pull request and generated 1 comment.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
…d resolve failing tests
This pull request introduces a new mapping lookup table (LUT) feature for ontology term translation, along with supporting utilities, API methods, and a demonstration notebook. It also updates pre-commit hooks and adds a new dependency. These changes make it easier to map and work with ontology terms across different databases.
Mapping LUT Feature and Utilities:
MappingLUTclass to represent mapping lookup tables for ontology IDs and database names, with utility methods for accessing mappings and database lists (ontograph/models.py).ontograph/utils.py).Client API Enhancements:
ClientOntologyfor loading mapping files, retrieving mapping LUTs, listing database names, and batch term retrieval (ontograph/client.py). [1] [2] [3] [4]ontograph/queries/navigator.py).Demonstration and Documentation:
chebi_mix_ontology.ipynbdemonstrating the mapping LUT workflow and use cases.Dependency and Tooling Updates:
pandasas a dependency for mapping file operations (pyproject.toml).ruff-pre-commit,pre-commit-hooks, andblacken-docsin.pre-commit-config.yaml. [1] [2] [3]