BenBrostoff / draftfast

A tool to automate and optimize DraftKings and FanDuel lineup construction.
277 stars 113 forks source link

Fan Duel CSV Upload #197

Closed Jzilllah closed 1 year ago

Jzilllah commented 1 year ago

Very new to coding here, so this may be a simple issue, but im having trouble importing Fan duel CSV.

Here is the error im getting:

Screenshot (261)

BenBrostoff commented 1 year ago

Can you attach the CSV you're attempting to ingest? It looks like the header doesn't have Position as a column name.

Jzilllah commented 1 year ago

both csv's i used appear to have position.

I used the below sheet for this attempt:

FDweek2.csv

I read in a previous comment that it hasnt been designed for the lineup build csv so i also tried with this version and got a different error. I have attached that below as well.

fdplayersweek2.csv

Screenshot (262)

I'll keep trying things out to see if I can fix it it may just be a simple error that i'm missing.

Thanks, J

BenBrostoff commented 1 year ago

I think you need to invoke game with either DRAFT_KINGS or FAN_DUEL - generate_players_from_csv expects a str instead of a Ruleset. You can see an example here https://github.com/BenBrostoff/draftfast/blob/b3278d0913e85084394f047fb1bfd6808e56c6b7/draftfast/test/test_optimize.py#L102-L114

ramGoli commented 1 year ago

Nevermind, you need to do this:

game=rules.FAN_DUEL, instead of game=rules.FD_NFL_RULE_SET

On Wed, Sep 14, 2022 at 8:30 AM Ram Goli @.***> wrote:

Try this path instead: salary_file_location="C://Users//J//Desktop//fdplayersweek2.csv"

On Wed, Sep 14, 2022 at 8:27 AM Ben Brostoff @.***> wrote:

I think you need to invoke game with either DRAFT_KINGS or FAN_DUEL - generate_players_from_csv expects a str instead of a Ruleset. You can see an example here https://github.com/BenBrostoff/draftfast/blob/b3278d0913e85084394f047fb1bfd6808e56c6b7/draftfast/test/test_optimize.py#L102-L114

— Reply to this email directly, view it on GitHub https://github.com/BenBrostoff/draftfast/issues/197#issuecomment-1246689627, or unsubscribe https://github.com/notifications/unsubscribe-auth/AF6P2UVAEWTNY2BKCYW5FBTV6HAD3ANCNFSM6AAAAAAQL44SDA . You are receiving this because you are subscribed to this thread.Message ID: @.***>

Jzilllah commented 1 year ago

It worked. Thank y'all so much! Will likely have more questions as I work to build my tests out, greatly appreciate this.