-
Notifications
You must be signed in to change notification settings - Fork 2
Fixed #85 from issues/task tab. Added a network construction guide. #86
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
Conversation
There was a problem hiding this 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 addresses issue #85 by adding a network construction guide to the documentation and making related improvements to the network construction functionality. The changes enhance the documentation structure and improve the graph construction functions.
Key Changes
- Added a network construction guide to the documentation structure
- Updated graph construction functions with improved parameter defaults and enhanced documentation
- Modified existing notebooks to improve clarity and remove redundant content
Reviewed Changes
Copilot reviewed 5 out of 8 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/source/index.rst | Added network construction guide to documentation toctree |
| docs/source/datasets.ipynb | Cleaned up output format and removed redundant network generation section |
| docs/source/Quick_Start.ipynb | Enhanced network construction section with detailed examples and updated formatting |
| bioneuralnet/utils/graph.py | Major refactor of all graph construction functions with improved documentation, parameter changes, and validation |
| bioneuralnet/metrics/plot.py | Added layout parameter to plot_network function with multiple layout options |
|
This is a "rushed" PR to address issue #85. |
|
Merging PR without review to update docs and package. All tests are passing. Will need to be reviewed over the weekend. @abdelhafizm |
This PR directly addresses #85