Add pytables recipe from defaults channel
Pytables recipe from the defaults channel.
Reason: The pytables packages from the defaults channel, for python 3.7, are not compatible with bob's version of hdf5 (1.10.6). There are compatible version for python 3.8 and 3.9, so it will be possible to drop this once we move to python 3.8+
Merge request reports
Activity
I don't know what to do with the
pip
andcython
dependencies. How does bob handle those ?They are using
mock
for testing the package, but bob's using nosetest or py-test ? Should we try to change that ?Also can you confirm that the
lzo
bzip2
numexpr
blosc
dependencies should be added to theconda_build_config.yaml
?Edited by Vincent POLLETNot sure what goes wrong with the macos build. Looks like permission issues ? https://gitlab.idiap.ch/bob/conda/-/jobs/221762#L142
Edited by Vincent POLLETI don't know what to do with the
pip
andcython
dependencies. How does bob handle those ? They are usingmock
for testing the package, but bob's using nosetest or py-test ? Should we try to change that ?I don't get it. Is this question relevant to this MR? Who are they?
Also can you confirm that the
lzo
bzip2
numexpr
blosc
dependencies should be added to theconda_build_config.yaml
?yes.
You're getting segmentation faults on mac builds. This issue is being tracked in nightlies#59 (closed)
Edited by Amir MOHAMMADII don't know what to do with the
pip
andcython
dependencies. How does bob handle those ? They are usingmock
for testing the package, but bob's using nosetest or py-test ? Should we try to change that ?I don't get it. Is this question relevant to this MR? Who are they?
pip
andcython
are just dependencies of the pytables package: https://gitlab.idiap.ch/bob/conda/-/blob/pytables/conda/pytables/meta.yaml#L32I was wondering if there was some special treatment for these specific dependencies, or if we just add them to
conda_build_config.yaml
About
mock
: it is also a dependency of the package, and it is used to test it after the build. Since bob's package are using different testing libraries, I was wondering we'd rather change it.
mentioned in commit d09e51ca