Merged
Conversation
…efactoring Sample dataset and gnn models refactoring, preparing for beta release version 2
Solve merge conflicts before release
BioInformatics - Stable release 1.0 Pre-commit failed because of an issue during R installation. I think I will be removing this from the pre-commit workflow since it is an optional component and not core of the package. Everything else passed. I tested all the tests/ locally as well and got no errors. Green light to merge
Removed import from jupiternotebook
…commit, locally It runs no issues all tests pass
…mmit Fixed precommit errors
There was a problem hiding this comment.
Pull Request Overview
This PR addresses a bug fix pre release while also introducing several enhancements and refactors across multiple modules and documentation. Key changes include improvements in subject representation error handling and messaging, updates to clustering modules with GPU seeding support, and updates to dataset loading logic, documentation, and CI workflows.
Reviewed Changes
Copilot reviewed 150 out of 156 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| bioneuralnet/downstream_task/subject_representation.py | Added GraphEmbedding implementation and improved error messages. |
| bioneuralnet/downstream_task/dpmon.py | Updated import usage and streamlined module structure. |
| bioneuralnet/downstream_task/init.py | Updated public API to include GraphEmbedding. |
| bioneuralnet/datasets/tcga_brca/README.md | Added detailed data preprocessing and file structure summary. |
| bioneuralnet/datasets/dataset_loader.py | Updated file loading logic using pathlib and multi-file support. |
| bioneuralnet/clustering/hybrid_louvain.py | Integrated GPU seed handling and enhanced logging. |
| bioneuralnet/clustering/correlated_pagerank.py | Enhanced tuning, GPU seeding, and error retry for PageRank. |
| bioneuralnet/clustering/correlated_louvain.py | Added GPU and seed support in tuning routines. |
| bioneuralnet/init.py | Updated version and expanded public API. |
| README.md | Revised project overview, structure, and installation instructions. |
| Cancer_example.ipynb | Added a new example notebook for demonstration. |
| CHANGELOG.md | Updated release notes with latest version details and fixes. |
| .readthedocs.yaml | Minor documentation configuration update. |
| .github/workflows/pre-commit.yml | Updated CI workflow with pre-commit upgrade and installation steps. |
Files not reviewed (6)
- .gitattributes: Language not supported
- .pre-commit-config.yaml: Language not supported
- MANIFEST.in: Language not supported
- bioneuralnet/datasets/monet/clinical_data.csv: Language not supported
- bioneuralnet/datasets/monet/phenotype.csv: Language not supported
- bioneuralnet/external_tools/SmCCNet.R: Language not supported
Comment on lines
+19
to
+20
| from ray.air import session | ||
|
|
There was a problem hiding this comment.
[nitpick] Duplicate import of 'session' from 'ray.air' detected. Removing the redundant import can clean up the code.
Suggested change
| from ray.air import session |
SundousHussein
approved these changes
Apr 24, 2025
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
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.
Bug fix pre release