Skip to content

Add find missing genes function and update gene map#11

Merged
rm1113 merged 6 commits into
mainfrom
feat/find_missing_genes
Nov 24, 2025
Merged

Add find missing genes function and update gene map#11
rm1113 merged 6 commits into
mainfrom
feat/find_missing_genes

Conversation

@rm1113
Copy link
Copy Markdown
Contributor

@rm1113 rm1113 commented Nov 24, 2025

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new public method find_missing_genes() to the UploadValidator class to help users identify genes in their AnnData files that are missing from the gene map. The implementation refactors existing validation logic to return missing gene information.

Key changes:

  • Added find_missing_genes() public method that returns a DataFrame of missing genes
  • Refactored _check_var_index() and _validate_gene_ids() to return boolean masks of missing genes instead of just validating
  • Added Union type hint import for optional return values

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 7 comments.

File Description
src/cap_upload_validator/upload_validator.py Added new find_missing_genes() method and refactored validation methods to return missing gene masks while maintaining backward compatibility with validation flow
CHANGELOG.md Documented the new find_missing_genes() function and gene map updates

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/cap_upload_validator/upload_validator.py Outdated
Comment thread src/cap_upload_validator/upload_validator.py Outdated
Comment thread src/cap_upload_validator/upload_validator.py Outdated
Comment thread src/cap_upload_validator/upload_validator.py Outdated
Comment thread src/cap_upload_validator/upload_validator.py Outdated
Comment thread src/cap_upload_validator/upload_validator.py
Comment thread src/cap_upload_validator/upload_validator.py
@rm1113 rm1113 merged commit fb6c191 into main Nov 24, 2025
5 checks passed
@rm1113 rm1113 deleted the feat/find_missing_genes branch November 24, 2025 14:25
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.

2 participants