Skip to content
This repository has been archived by the owner on Apr 12, 2021. It is now read-only.

Commit

Permalink
pythonPackages.accupy: remove perfplot from checkInputs
Browse files Browse the repository at this point in the history
they were only used for performance tests that aren't useful to us -
disabling these tests allows us to decouple from an unnecessary build
dependency.

convert to pytestCheckHook.
  • Loading branch information
risicle authored and Jon committed Sep 26, 2020
1 parent bcd728c commit 99175cf
Showing 1 changed file with 10 additions and 6 deletions.
16 changes: 10 additions & 6 deletions pkgs/development/python-modules/accupy/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,8 @@
, pybind11
, pyfma
, eigen
, pytest
, pytestCheckHook
, matplotlib
, perfplot
, isPy27
}:

Expand All @@ -35,9 +34,8 @@ buildPythonPackage rec {
];

checkInputs = [
pytest
pytestCheckHook
matplotlib
perfplot
];

postConfigure = ''
Expand All @@ -49,9 +47,15 @@ buildPythonPackage rec {
export HOME=$(mktemp -d)
'';

checkPhase = ''
pytest test
# performance tests aren't useful to us and disabling them allows us to
# decouple ourselves from an unnecessary build dep
preCheck = ''
for f in test/test*.py ; do
substituteInPlace $f --replace 'import perfplot' ""
done
'';
disabledTests = [ "test_speed_comparison1" "test_speed_comparison2" ];
pythonImportsCheck = [ "accupy" ];

meta = with lib; {
description = "Accurate sums and dot products for Python";
Expand Down

0 comments on commit 99175cf

Please sign in to comment.