-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
create a development environment #10995
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -286,6 +286,14 @@ cytoolz = "*" | |
| [feature.release.tasks] | ||
| release-contributors = "python ci/release_contributors.py" | ||
|
|
||
| [feature.dev.dependencies] | ||
| ipython = ">=9.8.0,<10" | ||
| black = ">=25.1.0,<26" | ||
| ipdb = ">=0.13.13,<0.14" | ||
|
|
||
| [feature.dev.pypi-dependencies] | ||
| pytest-accept = ">=0.2.2, <0.3" | ||
|
|
||
| [environments] | ||
| # Testing | ||
| # test-just-xarray = { features = ["test"] } # https://github.com/pydata/xarray/pull/10888/files#r2511336147 | ||
|
|
@@ -386,3 +394,14 @@ doc = { features = [ | |
| ] } | ||
| pre-commit = { features = ["pre-commit"], no-default-feature = true } | ||
| release = { features = ["release"], no-default-feature = true } | ||
| dev = { features = [ | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If you see this env as the "default" env for users, you can just rename it to new-default = []Also, you can use typing = { features = ["typing"], solve-group = "main" }
doc = { features = [
"doc",
"backends",
"test",
"accel",
"viz",
"extras",
], solve-group = "main"}
dev = { features = [ "py313",
"test",
"backends",
"accel",
"numba",
"dask",
"viz",
"extras",
"dev",
], solve-group = "main"}If this works, pixi will guarantee that they all use the exact same python and other base dependencies. When they conflict in dependencies, you can still leave out the solve-group.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks for taking the time to look at this @ruben-arts ! (and for responding to my Discord comment about this :)) ) . Also useful for another project I'm working on |
||
| "py313", | ||
| "test", | ||
| "backends", | ||
| "accel", | ||
| "numba", | ||
| "dask", | ||
| "viz", | ||
| "extras", | ||
| "dev", | ||
| ] } | ||
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've started using pdbpp now, and really like it.
Here's a list from my personal dev tool yaml. Can we add
pdbppline_profilermemory_profilermemraysnakevizicecreamipykernelsnoop(haven't tried this one yet, had forgotten about it but it looks great)Is
asvin thedevenv?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm... looks like it might be better to have a profiling feature / env, in that case. I'd probably include
pyinstrumentin that list, as well.memory_profilerdoes not appear to be maintained anymore (not sure if that's a problem).I don't think
asvis anywhere yet, I believe. Is it possible to use that withpixi, or does that have its own dependency installation system? I think I remember it usingcondaenvs before.Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not just one big fat one? :P
Interesting. I still find
%mprun -fin a notebook is very effective (recent example).I think it still manages its own; but you can tell it to use
rattleras a solver. I think it's useful to jsut addasvso we can run benchmarks.