feat: add mini neural network#7
Conversation
|
make sure you pass commit convention. I've pushed my neural network code. Could you move yours over to the same folder? it's under |
|
i believe i fixed the merge conflicts you will just have pull locally to make sure the changes are updated edit: i think i broke uv.lock. you will have to rebuild it and repush it. preferably dont commit uv.lock at all, if you could delete from the tracking that would be ideal |
JonathanPLev
left a comment
There was a problem hiding this comment.
could you do two more things:
- can you put all your files in a subdirectory in src/mini_nn, with your name on it, so we know its your nn
- can you put results in your readme so we can see how well your model perfroemd
|
and could you put your actual model into the |
JonathanPLev
left a comment
There was a problem hiding this comment.
can you make sure you fix the PR stuff, and then can you also reset uv.lock to how it was or remove it from your commit, and remove .DS_Store. thank you!
Nidhi mini-nnPredicts NBA players beating projected line (over/under) using a neural network Performance ResultsTraining Configuration
Results
Classification ReportConfusion Matric:[[768 658] Details
|
c5a1b21 to
8198ec7
Compare
bfc4619 to
0376974
Compare
What I changed
Why I changed it
How did I test it
Related Issues
Note: The PR title will become the commit message when squash merged, so make sure it follows conventional commit format!