Closed haguettaz closed 3 weeks ago
[!CAUTION]
Review failed
The pull request is closed.
The changes in this pull request involve a complete overhaul of the main
function in src/main.rs
, removing all neural network initialization and handling code in favor of a simple console output. Additionally, significant modifications were made to the Network
struct in src/network/network.rs
, where several methods were updated to accept distribution parameters instead of fixed ranges, enhancing flexibility in network initialization. The changes also included updates to test cases to accommodate the new method signatures and validate behavior with distribution parameters.
File | Change Summary |
---|---|
src/main.rs | Removed the entire implementation of the main function, which included neural network initialization and handling. Replaced with a simplified function that only prints "Hello world". Updated function signature to fn main() . |
src/network/network.rs | Updated method signatures for new_random , new_random_fout , new_random_fin , and new_random_fin_fout to accept distribution parameters instead of range parameters. Commented out previous implementation using ranges. |
src/network/network.rs
regarding the new_random
method and its parameters are directly related to the modifications in the main PR, which also involves the Network
struct and its initialization logic.🐇 In a world of neurons once bright,
Now just "Hello" takes flight.
With distributions set to play,
Our network's new in a simpler way.
Hopping through code, we cheer and sing,
For changes that make our hearts take wing! 🐇
Summary by CodeRabbit
New Features
Bug Fixes
Documentation