Hardeepex / NBA-Betting

0 stars 0 forks source link

sweep: how i can integrate the AI in this NBA Betting tool #5

Closed Hardeepex closed 10 months ago

Hardeepex commented 10 months ago
Checklist - [X] Create `src/main.py` ✓ https://github.com/Hardeepex/NBA-Betting/commit/5be90411f07f9d00eefa6b5edc74f1a37dc8859d [Edit](https://github.com/Hardeepex/NBA-Betting/edit/sweep/how_i_can_integrate_the_ai_in_this_nba_b/src/main.py#L1-L100) - [X] Running GitHub Actions for `src/main.py` ✓ [Edit](https://github.com/Hardeepex/NBA-Betting/edit/sweep/how_i_can_integrate_the_ai_in_this_nba_b/src/main.py#L1-L100) - [X] Modify `Flask/app.py` ✓ https://github.com/Hardeepex/NBA-Betting/commit/e25d6042c1b21a0de5b6fb8d94cdcafca94a29a3 [Edit](https://github.com/Hardeepex/NBA-Betting/edit/sweep/how_i_can_integrate_the_ai_in_this_nba_b/Flask/app.py#L1-L100) - [X] Running GitHub Actions for `Flask/app.py` ✓ [Edit](https://github.com/Hardeepex/NBA-Betting/edit/sweep/how_i_can_integrate_the_ai_in_this_nba_b/Flask/app.py#L1-L100) - [X] Create `tests/test_integration.py` ✓ https://github.com/Hardeepex/NBA-Betting/commit/32e8984d52e229add53faaa042e7e2b5fbb5ea7e [Edit](https://github.com/Hardeepex/NBA-Betting/edit/sweep/how_i_can_integrate_the_ai_in_this_nba_b/tests/test_integration.py) - [X] Running GitHub Actions for `tests/test_integration.py` ✓ [Edit](https://github.com/Hardeepex/NBA-Betting/edit/sweep/how_i_can_integrate_the_ai_in_this_nba_b/tests/test_integration.py)
sweep-ai[bot] commented 10 months ago

🚀 Here's the PR! #6

See Sweep's progress at the progress dashboard!
💎 Sweep Pro: I'm using GPT-4. You have unlimited GPT-4 tickets. (tracking ID: 794f7567fd)
Install Sweep Configs: Pull Request

Actions (click)

Sandbox Execution ✓

Here are the sandbox execution logs prior to making any changes:

Sandbox logs for 1b6fb67
Checking Flask/app.py for syntax errors... ✅ Flask/app.py has no syntax errors! 1/1 ✓
Checking Flask/app.py for syntax errors...
✅ Flask/app.py has no syntax errors!

Sandbox passed on the latest LogisticRegression, so sandbox checks will be enabled for this issue.


Step 1: 🔎 Searching

I found the following snippets in your repository. I will now analyze these snippets and come up with a plan.

Some code snippets I think are relevant in decreasing order of relevance (click to expand). If some file is missing from here, you can mention the path in the ticket description. https://github.com/Hardeepex/NBA-Betting/blob/1b6fb677c63f033a34617136d57dd64c7fd5aab4/README.md#L1-L61 https://github.com/Hardeepex/NBA-Betting/blob/1b6fb677c63f033a34617136d57dd64c7fd5aab4/notes.txt#L1-L67 https://github.com/Hardeepex/NBA-Betting/blob/1b6fb677c63f033a34617136d57dd64c7fd5aab4/src/Utils/tools.py#L1-L75 https://github.com/Hardeepex/NBA-Betting/blob/1b6fb677c63f033a34617136d57dd64c7fd5aab4/src/DataProviders/SbrOddsProvider.py#L1-L44

Step 2: ⌨️ Coding

Ran GitHub Actions for 5be90411f07f9d00eefa6b5edc74f1a37dc8859d:

--- 
+++ 
@@ -1,3 +1,4 @@
+import main
 from datetime import date
 import json
 from flask import Flask, render_template
@@ -68,4 +69,5 @@
     draftkings = fetch_draftkings(ttl_hash=get_ttl_hash())
     betmgm = fetch_betmgm(ttl_hash=get_ttl_hash())

-    return render_template('index.html', today=date.today(), data={"fanduel": fanduel, "draftkings": draftkings, "betmgm": betmgm})+    ai_predictions = main.get_ai_predictions()
+    return render_template('index.html', today=date.today(), data={"fanduel": fanduel, "draftkings": draftkings, "betmgm": betmgm, "ai_predictions": ai_predictions})

Ran GitHub Actions for e25d6042c1b21a0de5b6fb8d94cdcafca94a29a3:

Ran GitHub Actions for 32e8984d52e229add53faaa042e7e2b5fbb5ea7e:


Step 3: 🔁 Code Review

I have finished reviewing the code for completeness. I did not find errors for sweep/how_i_can_integrate_the_ai_in_this_nba_b.


🎉 Latest improvements to Sweep:


💡 To recreate the pull request edit the issue title or description. To tweak the pull request, leave a comment on the pull request. Join Our Discord