Il 21/08/22 13:59, Francesc Alted ha scritto:
> I am in the process of implementing support for Blosc2 (see
> direct-chunking-blosc2 branch). The plan is to use the wheels for
> python-blosc2 so as to avoid the costly task of compiling Blosc2 inside
> PyTables; this is also a much cleaner procedure. However, I need to
> install the packages in requirements.txt prior to execute setup.py, but
> apparently this is not possible (yet).
> I propose to modify the ci.yaml file to add a line similar to:
> Do you think there should be any issue in doing that?
the only problem that I see is that ci.yaml uses a conda environment do,
in principle, we should use the c-blosc2 package available in conda-forge.
Is that an issue for you?
In any case as a temporary solution IMHO using a mixed environment with
python-blosc2 installed with pip is not a big issue.
> Also, I tried to modify the files in .github/workflows (in the branch) but
> I am getting this:
> To https://github.com/PyTables/PyTables.git
> ! [remote rejected] direct-chunking-blosc2 -> direct-chunking-blosc2
> (refusing to allow a Personal Access Token to create or update workflow
> `.github/workflows/ci.yml` without `workflow` scope)
> error: failed to push some refs to 'https://github.com/PyTables/PyTables.git
Yes you should use an access token with the appropriate authorization
scopes. In the specific case it seems that the one that you are using
now is missing the "workflow" one.