Deprecate and undocument install_ext #8601

Merged
merged 1 commit into from Jul 15, 2015

Projects

None yet

4 participants

@Carreau
Member
Carreau commented Jul 2, 2015

It seem to me that Python packaging has come up to a point where
actually telling peoples they can install python extensions not using
package manager is unreasonable.

Recent extension even recommend using install_ext as the way to
install things, which is unsecure, confusing, and there is no update
mechanisme, nor version conflict resolution.


As I think this might be historical I would like @fperez input.

@Carreau Carreau Deprecate and undocument install_ext
It seem to me that Python packaging has come up to a point where
actually telling peoples they can install python extensions not using
package manager is unreasonable.

Recent extension even recommend using `install_ext` as **the way** to
install things, which is unsecure, confusing, and there is no update
mechanisme, nor version conflict resolution.
7021597
@Carreau
Member
Carreau commented Jul 9, 2015

Any feedback?

@minrk
Member
minrk commented Jul 9, 2015

I'm fine with this. I'd also be fine with de-emphasizing it without the deprecation message. I'd like to hear from @ellisonbg or @fperez.

@Carreau perhaps pinging the list for comment from people who might be using %load_ext?

@fperez
Member
fperez commented Jul 9, 2015

Why don't we try to get some input on the list? We might be surprised by what we hear, who knows...

@fperez
Member
fperez commented Jul 9, 2015

Though in principle I agree, the less we invent our own homebrewed, one-of-a-kind installation tools, the better...

@Carreau
Member
Carreau commented Jul 9, 2015 edited

@Carreau perhaps pinging the list for comment from people who might be using %load_ext?

I think you mean %install_ext . I'll ping the list.

[Edit 1 year later] poll on ML [/edit]

@fperez
Member
fperez commented Jul 10, 2015

There seems to be good consensus on the list on this, so +1 on moving forward. Thanks for taking the time to communicate it more broadly, always good to get the bigger picture, sometimes we get surprised...

@minrk
Member
minrk commented Jul 10, 2015

@fperez sounds good. Want to do the honors of hitting the green button?

@moorepants

Just wanted to point out this extension:

http://nbviewer.ipython.org/github/jrjohansson/version_information/blob/master/example.ipynb

It is not a installable package and I have this in probably 50+ notebooks now, all using %install_ext. The repo only has 16 stars so maybe it isn't in wide use though.

@Carreau
Member
Carreau commented Jul 10, 2015

@moorepants that exactly the kind of things that we want to avoid. It would take less than 5 minutes to make that a package, and wouldn't change the way to load the extension.

@moorepants

That's fine if you want to avoid it. I just mentioned it because your change will break all of my notebooks. This is just an example of a potential wider use case of %install_ext for you all to be aware of.

@Carreau
Member
Carreau commented Jul 10, 2015

I'm not sure in what this would break all your notebooks. Install ext should in anyway not be use at the start of all notebooks ? no? Once it's installed. It's installed, the %load_ext will still work right ?

@moorepants

Oh, I didn't know that. I just copied the example notebook I shared, so %install_ext is in every single notebook. Not sure what other people did. I've never looked at what %install_ext actually does.

@Carreau
Member
Carreau commented Jul 10, 2015 edited

Well actually install_ext is just downloading a file and putting it on PYTHONPATH at first order. If version_information.py is a package which is pip-installable, then you can replace %install_ext <url> by pip install <pacakgename> and the rest works the same.

@moorepants

Ok, I'll submit a PR to that package to make it pip installable. Thanks for explaining.

@moorepants moorepants referenced this pull request in jrjohansson/version_information Jul 10, 2015
Closed

IPython is deprecating %install_ext #9

@Carreau Carreau merged commit 068f9d2 into ipython:master Jul 15, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Carreau Carreau deleted the Carreau:no-install-ext branch Jul 15, 2015
@minrk minrk modified the milestone: 4.0 Aug 11, 2015
@Carreau Carreau referenced this pull request Aug 25, 2016
Closed

remove %install_ext #8634

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment