alan-turing-institute / AIrsenal

Machine learning Fantasy Premier League team
MIT License
296 stars 86 forks source link

Set lineup team_id None #653

Open rohanday3 opened 1 year ago

rohanday3 commented 1 year ago

Make transfers finds the team_id but set_lineup doesnt make changes and shows team_id as None. Should be an easy fix. Version 1.7.5

jack89roberts commented 1 year ago

Could you copy in the whole error message, if you're getting one? And does it make a difference if you explicitly set the team ID (airsenal_set_lineup --fpl_team_id <your_team_id>)? It's working for me so unsure what's going on.

rohanday3 commented 1 year ago

There is no error message but it's definitely a bug as the team_id is present for make transfers but not for set lineup. If I set it explicitly it does work.

(airsenalenv) rohan@MobyDick:~$ airsenal_make_transfers Login attempt 1/3... Failed
Login attempt 2/3... Logged in successfully
Getting starting squad for 810358 <------------ SEE HERE

Transfers to apply for fpl_team_id: 810358 for gameweek: 13
 ===========================================================

Bank Balance Before transfers is: £0.5
+--------+-----------------------+-------+
| Status |          Name         | Price |
+--------+-----------------------+-------+
|  OUT   | Alisson Ramses Becker |  £5.5 |
|   IN   |     Sam Johnstone     |  £4.6 |
+--------+-----------------------+-------+
Bank Balance After transfers is: £1.4

Apply Transfers? There is no turning back! (yes/no)no
Not applying transfers.  Can still choose starting 11 and captain.
fpl_team_id is None <-------------------- SEE HERE

Login attempt 1/3... Failed
Login attempt 2/3... Logged in successfully

=== starting 11 ===

== GK ==

David Raya Martin (ARS)

== DEF ==

Pedro Porro (TOT)
Vladimír Coufal (WHU)
William Saliba (ARS)

== MID ==

Mohamed Salah (LIV)
James Ward-Prowse (WHU)
Jarrod Bowen (WHU)(VC)
Hwang Hee-chan (WOL)

== FWD ==

Erling Haaland (MCI)(C)
Julián Álvarez (MCI)
Yoane Wissa (BRE)

=== subs ===

Alexis Mac Allister (LIV)
Alisson Ramses Becker (LIV)
Pau Torres (AVL)
Gabriel dos Santos Magalhães (ARS)

Apply changes to lineup? (yes/no)
jack89roberts commented 1 year ago

Does the lineup get applied correctly if you say yes? I think it's only a bug in what's printed, as it looks like that line is printed before we find your default team ID if one hasn't been passed in explicitly.

jack89roberts commented 1 year ago

Or you can try pulling and installing AIrsenal from the latest develop branch, as I've just pushed a fix for the printed message.

rohanday3 commented 1 year ago

I will try again next week to make sure but the lineup was not applied correctly this week.