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
python3Packages.tables: fix usage, add tests #119258
python3Packages.tables: fix usage, add tests #119258
Conversation
I miss understood that. Yeah, changing hdf5_1_10 would be an option. |
Result of 59 packages marked as broken and skipped:
1 package failed to build:307 packages skipped due to time constraints:
32 packages built successfully:
1 suggestion:
Note that build failures may predate this PR, and could be nondeterministic or hardware dependent. |
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 combine this PR into 2 or maybe 3 commits.
b4232ca
to
c588ada
Compare
* add myself as maintainer * update homepage * incorporate minor patches * add working tests
This package only exists for compatibility with ``python3Packages.tables``, as far as I can tell based on usage in nixpkgs. ``tables`` tests fail on hdf5 > v1.10.5, so we are reverting this to the last "good" version of hdf5 that works with ``tables``. ``hdf5_1_10`` should hopefully be unpinned on the next release of ``python3Packages.tables``. PyTables/PyTables#845
c588ada
to
19acc79
Compare
@bcdarwin gave consent for rolling back hdf5 by a patch-version in #119572 (comment). |
This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 38 packages marked as broken and skipped:
58 packages failed to build and already failed to build on hydra master:
16 packages failed to build and are new build failures:
336 packages built:
|
Motivation for this change
While trying to open a HDF5 file via
python3Packages.pandas
, realized that pandas' HDF5 functionality was broken b/cpython3Packages.tables
wasn't importing.The main failure I was having was due to setuptools not being installed at runtime (couldn't import
pkg_resources
), but also noticed that tests weren't being run.Trying to run the tests found that
tables
wasn't compatible withhdf5 v1.10.7
(PyTables/PyTables#845).tables
seems to be the only package usinghdf5_1_10
(nixpkgs/pkgs/top-level/python-packages.nix
Lines 8232 to 8239 in 311ceed
hdf5_1_10
tov1.10.5
?I know this probably needs a little bit of work/cleanup, just wanted to get some opinions on this.
Things tried:
Improvements:
pythonImportsCheck
pytestCheckHook
)Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)