search

marcoblume / pinnacle.API

R Wrapper for the Pinnacle API
40 stars 13 forks source link

showOddsDF merge error #8

Closed TuomoNieminen closed 7 years ago

TuomoNieminen commented 7 years ago

Hi,

Thank you for providing this wrapper for the Pinnacle API. I tried the following:

odds <- pinnacle.API::showOddsDF(29, oddsformat = "AMERICAN")

And got this error.

Error in merge.data.table(., inrunning[sports.id %in% sportid], by.x = "leagues.events.id",  : 
  Elements listed in `by.y` must be valid column names in y.

I am using R 3.3.3 and pinnacle API 2.0.4

shapenaji commented 7 years ago

Hello!

This has been fixed as of v2.0.5 here on github.

You can install with the devtools package,

devtools::install_github('marcoblume/pinnacle.API')

Not yet on CRAN however

shapenaji commented 7 years ago

(The problem emerges if there is no Live Soccer Available)

I'll keep this open to see if this solves your problem

TuomoNieminen commented 7 years ago

Yes the development version seems to fix the problem for me. Thank you.