UniProt Parser Refactor – Improved Modularity, Identifier Extraction, and Test Coverage #45
+2,102
−1,563
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ialarmedalien
force-pushed
the
uniprot-refactor-v2
branch
2 times, most recently
from
3b89f65 to
2e45b47
Compare
ialarmedalien
force-pushed
the
uniprot-refactor-v2
branch
3 times, most recently
from
2a781b3 to
bba5e5a
Compare
![github-code-quality[bot]](https://avatars.githubusercontent.com/u/9919?s=60&v=4){kind=small}
alinakbase
force-pushed
the
uniprot-refactor-v2
branch
2 times, most recently
from
ec65f68 to
bfbf335
Compare
alinakbase
force-pushed
the
uniprot-refactor-v2
branch
from
a49eea1 to
8653e2c
Compare
| if os.path.exists(tmp_path): | ||
| try: | ||
| os.remove(tmp_path) | ||
| except Exception: |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #45 +/- ##
===========================================
+ Coverage 43.78% 44.92% +1.13%
===========================================
Files 42 44 +2
Lines 2592 2767 +175
===========================================
+ Hits 1135 1243 +108
- Misses 1457 1524 +67
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This refactor introduces a cleaner, modular, and more maintainable architecture for UniProt data parsing within the cdm_data_loader_utils package. The new design separates concerns across multiple parser components, centralizes shared identifier extraction, enhances XML utilities, and adds a comprehensive test suite to ensure long-term stability.
Key improvements include:
• Modular parser structure under cdm_data_loader_utils/parsers/
• Unified shared identifier extraction (shared_identifiers.py)
• Robust XML parsing utilities (xml_utils.py)
• Refactored UniProt parser (uniprot.py) with clearer logic paths
• Complete tests for UniProt refactor, including:
• shared identifiers
• XML utilities
• UniProt entry parsing
• Cleaner directory layout aligned with CDM conventions
This refactor provides a foundation for future expansion (features, evidence, associations, and publications) while improving maintainability and reducing duplicated logic.