lbenz730 / ncaahoopR

An R package for working with NCAA Basketball Play-by-Play Data
MIT License
198 stars 48 forks source link

Unable to get data for most of the functions #84

Closed jmichaelp62 closed 1 year ago

jmichaelp62 commented 1 year ago

Hi, I'm experiencing problems as detailed below. This started yesterday mid-morning. There had been no restart of R, when I returned to use it after a couple of hours I couldn't pull info. Previously all worked well. I tried reinstalling the ncaahoopR package & then the entire R software, both to no avail. I tried a different school, "Florida" & also a Florida game id 401483032, but that didn't work either. Thanks.

The following return an error

The following work: get_roster("Texas", "2022-23") get_schedule("Texas", "2022-23") get_game_ids("Texas", "2022-23")

lbenz730 commented 1 year ago

Thank you, I will certainly look into this.

lbenz730 commented 1 year ago

At first glance the root of this issue seems very major -- as in the whole package would have to be re-written...I'll see what I can do but I think it will take some time.

lbenz730 commented 1 year ago

Actually I lied I think it'll be fixable.

lbenz730 commented 1 year ago

I think I have figured out how to fix but may take a few days to get everything up and running. Thanks for alerting me and for your patience.

jmichaelp62 commented 1 year ago

Thanks so much for looking into this & figuring out a solution.

mtkearney commented 1 year ago

thanks for looking into this - excited to see what the solution is!

lbenz730 commented 1 year ago

Have a solution that seems to be working. Some testing to come tmrw but hopefully a release by end of week

jmichaelp62 commented 1 year ago

Thanks! I wish I knew how to help.

On Tue, Nov 22, 2022 at 8:41 PM Luke Benz @.***> wrote:

Have a solution that seems to be working. Some testing to come tmrw but hopefully a release by end of week

— Reply to this email directly, view it on GitHub https://github.com/lbenz730/ncaahoopR/issues/84#issuecomment-1324444406, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZAUDZRFKLKKDCROW76RPLTWJVY45ANCNFSM6AAAAAASEHK3OU . You are receiving this because you authored the thread.Message ID: @.***>

lbenz730 commented 1 year ago

Ok all the data functions should be working again. Please let me know if any issues persist.

mtkearney commented 1 year ago

Luke, thanks so much for the fix, it's working great! Slight improvement - FGA/FGM are coming through as characters, not doubles. I fixed using sapply, but just wanted to let you know in case it's something easy to fix on your end

lbenz730 commented 1 year ago

Fixed this so should come through as doubles now like the rest of them, if you re-install.