Skip to content

Improvements#2

Open
Dragjon wants to merge 6 commits intokidscoots101:mainfrom
Dragjon:main
Open

Improvements#2
Dragjon wants to merge 6 commits intokidscoots101:mainfrom
Dragjon:main

Conversation

@Dragjon
Copy link
Copy Markdown

@Dragjon Dragjon commented Jul 25, 2024

  • More data in dataset
  • Removed 'lr' = 0.001 which was causing problems
  • Changed rmsprop --> adam optimiser
  • Changed <= threshold to > threshold for the new model
  • Some minor changes to code
  • Changed threshold to 0.5 (the estimated threshold is not accurate)

Dragjon added 6 commits July 25, 2024 16:18
- More data in dataset
- Removed 'lr' = 0.001 which was causing problems
- Changed rmsprop --> adam optimiser
- Changed <= threshold to > threshold for the new model
- Some minor changes
@kidscoots101
Copy link
Copy Markdown
Owner

Hey Dragjon!

Thanks for your help in improving our algorithm. Really appreciate it :)

Hmmm was just wondering about what the rationale was between switching from RMSProp to Adam Optimiser?

Thanks!

@Dragjon
Copy link
Copy Markdown
Author

Dragjon commented Jul 31, 2024

I used the Adam optimiser instead as it is known to adapt better to different types of data taking the advantages of both Momentum and RMSProp, usually the first optimiser used is the optimiser as it requires little parameter tuning to achieve results

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants