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
Add python packages: pyensembl and its dependencies, gffutils, pyBigWig #87551
Conversation
Add config option services.xserver.windowManager.exwm.loadScript, which is passed to Emacs (as the -l option) to load after the user's init file.
nixos/exwm: allow custom Emacs load script
Please rebase, don't merge master. |
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.
Please ensure the modules are somehow tested. Maybe supporting the unit tests is too much, but we have the pythonImportsCheckHook
so you could do import checks at least.
Also, remove empty lists and strings.
Finally, will you maintain the new packages?
|
||
pybedtools = callPackage ../development/python-modules/pybedtools { }; | ||
|
||
pyBigWig = callPackage ../development/python-modules/pyBigWig { }; |
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.
normalized names (lowercase and dashes instead of underscores)
@FRidh thanks for pointing this out. The module 'pybigwig' already existed in the correct format (lowercase) so I just reverted that commit. I cannot guarantee to maintain the packages in a timely manner, but will try to update their versions from time to time. I'm rebasing to master now and will push then. New rebase is here: #89213 |
Motivation for this change
Both pyenseml and gffutils are commonly used python packages (> 100 stars on GitHub), so I added them to nix packages.
Things done
nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
) (only pyensembl has a supplemental binary)nix path-info -S
before and after)