Commit 937edb65 authored by Fabian Affolter's avatar Fabian Affolter Committed by github-actions[bot]
Browse files

python311Packages.phik: refactor, disable failing tests

(cherry picked from commit fc1c549b)
parent b44ebb5f
Loading
Loading
Loading
Loading
+44 −24
Original line number Diff line number Diff line
@@ -2,27 +2,29 @@
, buildPythonPackage
, cmake
, fetchFromGitHub
, isPy3k
, pytestCheckHook
, nbconvert
, joblib
, jupyter
, jupyter-client
, numpy
, scipy
, pandas
, matplotlib
, nbconvert
, ninja
, numba
, numpy
, pandas
, pybind11
, pytestCheckHook
, pythonOlder
, scikit-build
, scipy
, setuptools
}:

buildPythonPackage rec {
  pname = "phik";
  version = "0.12.3";
  disabled = !isPy3k;
  format = "pyproject";
  pyproject = true;

  disabled = pythonOlder "3.7";

  src = fetchFromGitHub {
    owner = "KaveIO";
@@ -31,11 +33,11 @@ buildPythonPackage rec {
    hash = "sha256-9o3EDhgmne2J1QfzjjNQc1mUcyCzoVrCnWXqjWkiZU0=";
  };

  nativeCheckInputs = [
    pytestCheckHook
    nbconvert
    jupyter
    jupyter-client
  nativeBuildInputs = [
    cmake
    ninja
    scikit-build
    setuptools
  ];

  propagatedBuildInputs = [
@@ -48,17 +50,20 @@ buildPythonPackage rec {
    pybind11
  ];

  # uses setuptools to drive build process
  nativeCheckInputs = [
    pytestCheckHook
    nbconvert
    jupyter
    jupyter-client
  ];

  # Uses setuptools to drive build process
  dontUseCmakeConfigure = true;

  nativeBuildInputs = [
    cmake
    ninja
    scikit-build
  pythonImportsCheck = [
    "phik"
  ];

  pythonImportsCheck = [ "phik" ];

  postInstall = ''
    rm -r $out/bin
  '';
@@ -68,12 +73,27 @@ buildPythonPackage rec {
    rm -r phik
  '';

  disabledTests = [
    # TypeError: 'numpy.float64' object cannot be interpreted as an integer
    # https://github.com/KaveIO/PhiK/issues/73
    "test_significance_matrix_hybrid"
    "test_significance_matrix_mc"
  ];

  disabledTestPaths = [
    # Don't test integrations
    "tests/phik_python/integration/"
  ];

  meta = with lib; {
    description = "Phi_K correlation analyzer library";
    longDescription = "Phi_K is a new and practical correlation coefficient based on several refinements to Pearson’s hypothesis test of independence of two variables.";
    homepage = "https://phik.readthedocs.io/en/latest/";
    changelog = "https://github.com/KaveIO/PhiK/blob/${src.rev}/CHANGES.rst";
    maintainers = with maintainers; [ melsigl ];
    longDescription = ''
      Phi_K is a new and practical correlation coefficient based on several refinements to
      Pearson’s hypothesis test of independence of two variables.
    '';
    homepage = "https://phik.readthedocs.io/";
    changelog = "https://github.com/KaveIO/PhiK/blob/${version}/CHANGES.rst";
    license = licenses.asl20;
    maintainers = with maintainers; [ melsigl ];
  };
}