-
-
Notifications
You must be signed in to change notification settings - Fork 13.8k
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
python312Packages.plotly: 5.23.0 -> 5.24.0 and some TLC #341141
Conversation
Result of 8 packages marked as broken and skipped:
3 packages failed to build:
115 packages built:
|
Both |
Result of 10 packages marked as broken and skipped:
118 packages built:
|
If you want to enable the checkPhase, please make sure it actually runs the tests. I enabled pytest and ran through a couple of the errors, but there are lots more to figure out :) diff --git a/pkgs/development/python-modules/plotly/default.nix b/pkgs/development/python-modules/plotly/default.nix
index 0ca2cc6c08a0..e94ce037258c 100644
--- a/pkgs/development/python-modules/plotly/default.nix
+++ b/pkgs/development/python-modules/plotly/default.nix
@@ -6,6 +6,20 @@
packaging,
tenacity,
kaleido,
+ pytestCheckHook,
+ pandas,
+ requests,
+ matplotlib,
+ xarray,
+ pillow,
+ scipy,
+ psutil,
+ statsmodels,
+ ipython,
+ which,
+ orca,
+ nbformat,
+ scikit-image,
}:
buildPythonPackage rec {
@@ -20,9 +34,9 @@ buildPythonPackage rec {
hash = "sha256-frSUybQxst4wG8g8U43Nay9dYCUXuR3dBealwPVyFdI=";
};
- postPatch = ''
- cd packages/python/plotly
+ sourceRoot = "${src.name}/packages/python/plotly";
+ postPatch = ''
substituteInPlace pyproject.toml \
--replace-fail "\"jupyterlab~=3.0;python_version>='3.6'\"," ""
'';
@@ -37,6 +51,23 @@ buildPythonPackage rec {
kaleido
];
+ nativeCheckInputs = [
+ pytestCheckHook
+ pandas
+ requests
+ matplotlib
+ xarray
+ pillow
+ scipy
+ psutil
+ statsmodels
+ ipython
+ which
+ orca
+ nbformat
+ scikit-image
+ ];
+
pythonImportsCheck = [ "plotly" ];
meta = { |
I enabled |
please review #347567
pythonImportsCheckHook is gated by doInstallCheck. :) |
Description of changes
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.