-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
spyder: withPackages passthru for python console (closes #70986) #71989
Conversation
Been a couple of weeks now. Wondering if you guys might have a quick chance to give it a quick go over before any merge incompatibilities creep in between it and HEAD. Thanks. 😃 |
Is this package meant to be used by other python packages? |
I don't believe so. It a scientific python IDE. The |
Then I would highly recommend removing it from python-modules and putting it into applications/scientific or something simliar |
You can use it as a library to e.g. embed the |
I'll have a look today |
interesting, I'm not super familiar with spyder, but if you did want to use it as a dependency in other packages, then I would recommend against doing the withPackages, as this might break composibility with other python packages. I will defer to @FRidh though, as he's more familiar with withPackages and it's implications |
I think this is no longer needed, see #70986 (comment). |
I think
while
|
My understanding of 6e9743f is that it addresses the fact that IPython wouldn't start, but it doesn't address the need to be able to specify the packages that are available in that IPython session (which would be whatever the code you are working on requires). |
imports work fine. E.g., if I add numpy to the env I can import it. |
Thank you for your contributions.
|
Existing way to do this was given at end of issue #70986. |
Motivation for this change
Address the issue in #70986. That is, the python process spyder spawned for the ipython console didn't have the required
spyder-kernels
module to interface withspyder
.It also builds a bit on top of that by adding a
withPackages
override that is essentially used in apython.withPackages
statement for building the python process that spyder spawns for the ipython console. This lets the user add whatever packages they need to work with their code.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @gebner