Skip to content
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

Updated ReadMe for future Use and Contributions #284

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ejmercier
Copy link

I refactored part of the readme to make it easier for new users to understand, install and contribute to the project.

These are the changes I made:

  • Separated the project description into an About section and a Betting Strategy section
  • Added an Installation section for creating an environment and installing packages
  • Separated getting new data and training model's section into two separate headers
  • Added a link to the open-source contributing guidelines for users to follow when contributing

README.md Outdated Show resolved Hide resolved
```

Odds data will be automatically fetched from sbrodds if the -odds option is provided with a sportsbook. Options include: fanduel, draftkings, betmgm, pointsbet, caesars, wynn, bet_rivers_ny
python3 main.py -xgb -odds=fanduel

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is xgb model exclusively used over the other ones?


Takes all team data from the 2007-08 season to current season, matched with odds of those games, using a neural network to predict winning bets for today's games.

Achieves ~75% accuracy on money lines and ~58% on under/overs.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this correct? The current model in the repo is at 68.9% on ML and 54.8% on OU

Co-authored-by: Kai Forsyth <hello@kaiforsyth.com>
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.

None yet

4 participants