-
Notifications
You must be signed in to change notification settings - Fork 63
Added celldancer, deepvelo, and latentvelo to dyn.tl.extvelo (#721)
#731
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
Merged
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
…ing steps - Replaced deprecated functions with Scanpy equivalents for log transformation, scaling, PCA, neighbors, and UMAP. - Updated `extvelo` function to create a temporary directory if it doesn't exist and added dynamics metadata to the AnnData object.
…loDataset - Integrated tqdm progress bar for epoch tracking in BaseTrainer. - Enhanced logging to display metrics during training based on progress bar usage. - Updated VeloDataset to handle sparse matrices and adjusted PCA solver. - Improved neighbor graph construction in VeloDataset for better performance.
- Integrated deepvelo training process into extvelo function. - Updated AnnData layers and metadata to accommodate deepvelo outputs. - Ensured compatibility with existing velocity calculations and data structures.
- Implemented velovi integration within extvelo function. - Added necessary data processing and metadata updates for velovi outputs. - Included comments indicating successful testing for existing methods and noted the need for testing on the new velovi method.
…onality - Included necessary imports for torch, numpy, and scipy to enhance the velovi method. - Ensured proper integration of the velovi setup process within the extvelo function. - Maintained existing structure and comments for clarity on method testing status.
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #731 +/- ##
==========================================
- Coverage 30.26% 28.11% -2.15%
==========================================
Files 258 295 +37
Lines 43407 46721 +3314
==========================================
Hits 13137 13137
- Misses 30270 33584 +3314 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
- Adjusted function signature for extvelo to enhance readability. - Added missing comma in the parameters dictionary for consistency. - Included additional import for data preprocessing to support velovi method. - Ensured adata is copied before processing to maintain data integrity.
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 pull request introduces the core infrastructure for integrating DeepVelo, Celldancer, and latentvelo
Example