search

zmjones / edarf

exploratory data analysis using random forests
MIT License
68 stars 11 forks source link

plot_twoway_partial: argument "grid" is missing #10

Closed danilofreire closed 9 years ago

danilofreire commented 9 years ago

Hi Zack!

Congratulations for your package! It's very indeed very convenient.

I was testing the code today and found a bug. I ran the examples you provided here and I got this message:

plot_twoway_partial(pd_reg$Education, pd_reg$pred, smooth = TRUE)
Error in noNA(grid) : argument "grid" is missing, with no default

I'm pretty sure it's an easy fix, but I'm not sure how to do it (and maybe you want to update the example too). I'm using Revolution R Open 8.0 and R 3.1.1 on Ubuntu 14.04, if that matters.

Thanks!

zmjones commented 9 years ago

Hey Daniel. I will check this bug out tomorrow or the next day. This package is in for a major re-write in the next week as well. Thanks for checking it out!

danilofreire commented 9 years ago

Sure, I'm glad to help. I'll wait for the next version, then. All the best!