Skip to content

Improvements#2

Open
Dragjon wants to merge 6 commits into
kidscoots101:mainfrom
Dragjon:main
Open

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

Conversation

@Dragjon

@Dragjon Dragjon commented Jul 25, 2024

Copy link
Copy Markdown
  • 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

Dragjon commented Jul 31, 2024

Copy link
Copy Markdown
Author

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