I think in other estimators we do not issue a warning when a sparse matrix conversion is required. The fact that `safe_as_array` chokes might be a bug, but should probably be fixed in a separate PR. Please add an inline comment such as `XXX: add support for other sparse matrix formats without data copy if possible`