search

giuseppec / iml

iml: interpretable machine learning R package
https://giuseppec.github.io/iml/
Other
492 stars 87 forks source link

[WIP] moves ggplot2 and keras to suggests #175

Closed christophM closed 3 years ago

pat-s commented 3 years ago

Why favor require() > requireNamespace()? In a package context, it is good practice to use requireNamespace() :)

OTOH there were quite a lot ggplot calls inside the package, so this might be quite an effort.

christophM commented 3 years ago

I am not sure if removing ggplot2 dependence is that valuable. As a user, I guess you would most of the time plot something with the iml package.