Make IPython an optional dependency

rkern / line_profiler

(OLD REPO) Line-by-line profiling for Python - Current repo ->

https://github.com/pyutils/line_profiler

Other

3.6k stars 254 forks source link

Make IPython an optional dependency #112

Open simonpercivall opened 6 years ago

simonpercivall commented 6 years ago

It would be great if line_profiler didn't always require IPython to install, instead making it either a soft dependency, or an extras_require (with graceful degradation, of course).

caethan commented 6 years ago

Yes, we would love to do this. I've been trying out ways to make it a soft dependency, but it's tricky with the way the IPython API has changed. If you've got any suggestions, let us know!

simonpercivall commented 6 years ago

In its most simple way, I'd do it like this: PR #114

'Course, it'd be better to split out into a separate file, to make it look a bit cleaner.