dfs-with-r / coach

Lineup optimization for daily fantasy sports
https://dfs-with-r.github.io/coach/
GNU General Public License v3.0
47 stars 14 forks source link

mylineups.csv DraftKings NFL needs extra commas in header #42

Open abm515 opened 1 year ago

abm515 commented 1 year ago

Hi! I've never contributed to a GitHub project. I think I found a glitch in DraftKings NFL where the mylineups.csv is writing QB,RB,RB,WR,WR,WR,TE,FLEX,DST where it needs to be QB,RB,RB,WR,WR,WR,TE,FLEX,DST,, in order to avoid an error. I imagine they changed the required format after you published.

Seems to me that we need to change this line in https://github.com/dfs-with-r/coach/blob/master/R/write.r pos_levels <- c("QB", "RB", "WR", "TE", "FLEX", "DST") to pos_levels <- c("QB", "RB", "WR", "TE", "FLEX", "DST","","") but I'm not 100% sure. I'd be happy to change the code and contribute, I've just never changed anything in GitHub so I may need a bit of hand-holding, and also would like to hear your opinion on if this would fix the issue.

Thanks! a

zamorarr commented 1 year ago

Hi @abm515 thanks for catching that! Yup I am happy to walk you through submitting this change. Do you happen to have an example file that DraftKings would accept? You can just attach it to a comment here.

As for submitting a change, can you try going to https://github.com/dfs-with-r/coach/blob/master/R/write.r and clicking the Edit icon (looks like a pencil)? You should be able to make a change and submit a Pull Request. If Github does not allow you to do that, you might have to Fork the repository first (icon looks like the Sharing icon).