Skip to content
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

awkward v2.6.8 #147

Merged
merged 4 commits into from
Sep 13, 2024
Merged

Conversation

regro-cf-autotick-bot
Copy link
Contributor

It is very likely that the current package version for this feedstock is out of date.

Checklist before merging this PR:

  • Dependencies have been updated if changed: see upstream
  • Tests have passed
  • Updated license if changed and license_file is packaged

Information about this PR:

  1. Feel free to push to the bot's branch to update this PR if needed.
  2. The bot will almost always only open one PR per version.
  3. The bot will stop issuing PRs if more than 3 version bump PRs generated by the bot are open. If you don't want to package a particular version please close the PR.
  4. If you want these PRs to be merged automatically, make an issue with @conda-forge-admin,please add bot automerge in the title and merge the resulting PR. This command will add our bot automerge feature to your feedstock.
  5. If this PR was opened in error or needs to be updated please add the bot-rerun label to this PR. The bot will close this PR and schedule another one. If you do not have permissions to add this label, you can use the phrase @conda-forge-admin, please rerun bot in a PR comment to have the conda-forge-admin add it for you.

Pending Dependency Version Updates

Here is a list of all the pending dependency version updates for this repo. Please double check all dependencies before merging.

Name Upstream Version Current Version
awkward-cpp 38 Anaconda-Server Badge

This PR was created by the regro-cf-autotick-bot. The regro-cf-autotick-bot is a service to automatically track the dependency graph, migrate packages, and propose package version updates for conda-forge. Feel free to drop us a line if there are any issues! This PR was generated by - please use this URL for debugging.

@conda-forge-webservices
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

I do have some suggestions for making it better though...

For recipe/meta.yaml:

  • Use importlib-metadata instead of importlib_metadata

@matthewfeickert
Copy link
Member

matthewfeickert commented Sep 13, 2024

edit: c.f. #147 (comment) for correct reason.


This is breaking unfortunately due to the following combination of things:

Here's an example of why this is breaking everything:

$ docker run --rm -ti debian:bookworm
root@bb6e803310c7:/# apt update && apt install -y curl
root@bb6e803310c7:/# curl -sLO https://github.com/conda-forge/miniforge/releases/latest/download/Miniforge3-Linux-x86_64.sh
root@bb6e803310c7:/# bash Miniforge3-Linux-x86_64.sh -h

usage: Miniforge3-Linux-x86_64.sh [options]

Installs Miniforge3 24.7.1-0

-b           run install in batch mode (without manual intervention),
             it is expected the license terms (if any) are agreed upon
-f           no error if install prefix already exists
-h           print this help message and exit
-p PREFIX    install prefix, defaults to /root/miniforge3, must not contain spaces.
-s           skip running pre/post-link/install scripts
-u           update an existing installation
-t           run package tests after installation (may install conda-build)

root@bb6e803310c7:/# bash Miniforge3-Linux-x86_64.sh  # answer yes to update shell profile
root@bb6e803310c7:/# . ~/.bashrc

Conda can create the environment using the optimizations

(base) root@bb6e803310c7:/# conda create --name conda-test 'awkward-cpp=38'
Channels:
 - conda-forge
Platform: linux-64
Collecting package metadata (repodata.json): done
Solving environment: done

## Package Plan ##

  environment location: /root/miniforge3/envs/conda-test

  added / updated specs:
    - awkward-cpp=38


The following NEW packages will be INSTALLED:

  _libgcc_mutex      conda-forge/linux-64::_libgcc_mutex-0.1-conda_forge 
  _openmp_mutex      conda-forge/linux-64::_openmp_mutex-4.5-2_gnu 
  _x86_64-microarch~ conda-forge/noarch::_x86_64-microarch-level-3-2_skylake 
  awkward-cpp        conda-forge/linux-64::awkward-cpp-38-py312h87d15fd_300 
  bzip2              conda-forge/linux-64::bzip2-1.0.8-h4bc722e_7 
  ca-certificates    conda-forge/linux-64::ca-certificates-2024.8.30-hbcca054_0 
  ld_impl_linux-64   conda-forge/linux-64::ld_impl_linux-64-2.40-hf3520f5_7 
  libblas            conda-forge/linux-64::libblas-3.9.0-23_linux64_openblas 
  libcblas           conda-forge/linux-64::libcblas-3.9.0-23_linux64_openblas 
  libexpat           conda-forge/linux-64::libexpat-2.6.3-h5888daf_0 
  libffi             conda-forge/linux-64::libffi-3.4.2-h7f98852_5 
  libgcc             conda-forge/linux-64::libgcc-14.1.0-h77fa898_1 
  libgcc-ng          conda-forge/linux-64::libgcc-ng-14.1.0-h69a702a_1 
  libgfortran        conda-forge/linux-64::libgfortran-14.1.0-h69a702a_1 
  libgfortran-ng     conda-forge/linux-64::libgfortran-ng-14.1.0-h69a702a_1 
  libgfortran5       conda-forge/linux-64::libgfortran5-14.1.0-hc5f4f2c_1 
  libgomp            conda-forge/linux-64::libgomp-14.1.0-h77fa898_1 
  liblapack          conda-forge/linux-64::liblapack-3.9.0-23_linux64_openblas 
  libnsl             conda-forge/linux-64::libnsl-2.0.1-hd590300_0 
  libopenblas        conda-forge/linux-64::libopenblas-0.3.27-pthreads_hac2b453_1 
  libsqlite          conda-forge/linux-64::libsqlite-3.46.1-hadc24fc_0 
  libstdcxx          conda-forge/linux-64::libstdcxx-14.1.0-hc0a3c3a_1 
  libuuid            conda-forge/linux-64::libuuid-2.38.1-h0b41bf4_0 
  libxcrypt          conda-forge/linux-64::libxcrypt-4.4.36-hd590300_1 
  libzlib            conda-forge/linux-64::libzlib-1.3.1-h4ab18f5_1 
  ncurses            conda-forge/linux-64::ncurses-6.5-he02047a_1 
  numpy              conda-forge/linux-64::numpy-2.1.1-py312h58c1407_0 
  openssl            conda-forge/linux-64::openssl-3.3.2-hb9d3cd8_0 
  pip                conda-forge/noarch::pip-24.2-pyh8b19718_1 
  python             conda-forge/linux-64::python-3.12.5-h2ad013b_0_cpython 
  python_abi         conda-forge/linux-64::python_abi-3.12-5_cp312 
  readline           conda-forge/linux-64::readline-8.2-h8228510_1 
  setuptools         conda-forge/noarch::setuptools-73.0.1-pyhd8ed1ab_0 
  tk                 conda-forge/linux-64::tk-8.6.13-noxft_h4845f30_101 
  tzdata             conda-forge/noarch::tzdata-2024a-h8827d51_1 
  wheel              conda-forge/noarch::wheel-0.44.0-pyhd8ed1ab_0 
  xz                 conda-forge/linux-64::xz-5.2.6-h166bdaf_0 


Proceed ([y]/n)? y


Downloading and Extracting Packages:

Preparing transaction: done
Verifying transaction: done
Executing transaction: done
#
# To activate this environment, use
#
#     $ conda activate conda-test
#
# To deactivate an active environment, use
#
#     $ conda deactivate

but mamba v1.5.9 can not given conda-forge/microarch-level-feedstock#10

(base) root@bb6e803310c7:/# mamba --version
mamba 1.5.9
conda 24.7.1
(base) root@bb6e803310c7:/# mamba create --name mamba-test 'awkward-cpp=38'

Looking for: ['awkward-cpp=38']

warning  libmamba Could not parse mod/etag header
warning  libmamba Could not parse mod/etag header
conda-forge/noarch                                  16.4MB @  21.2MB/s  0.8s
conda-forge/linux-64                                38.0MB @  38.6MB/s  1.1s
Could not solve for environment specs
The following packages are incompatible
└─ awkward-cpp 38**  is not installable because there are no viable options
   ├─ awkward-cpp 38 would require
   │  └─ _x86_64-microarch-level >=1  but there are no viable options
   │     ├─ _x86_64-microarch-level 1 would require
   │     │  └─ __archspec 1 core2, which is missing on the system;
   │     ├─ _x86_64-microarch-level 1 would require
   │     │  └─ __archspec 1 k10, which is missing on the system;
   │     ├─ _x86_64-microarch-level 1 would require
   │     │  └─ __archspec 1 nocona, which is missing on the system;
   │     ├─ _x86_64-microarch-level 1 would require
   │     │  └─ __archspec 1 x86_64, which is missing on the system;
   │     ├─ _x86_64-microarch-level 2 would require
   │     │  └─ __archspec 1 bulldozer, which is missing on the system;
   │     ├─ _x86_64-microarch-level 2 would require
   │     │  └─ __archspec 1 ivybridge, which is missing on the system;
   │     ├─ _x86_64-microarch-level 2 would require
   │     │  └─ __archspec 1 nehalem, which is missing on the system;
   │     ├─ _x86_64-microarch-level 2 would require
   │     │  └─ __archspec 1 piledriver, which is missing on the system;
   │     ├─ _x86_64-microarch-level 2 would require
   │     │  └─ __archspec 1 sandybridge, which is missing on the system;
   │     ├─ _x86_64-microarch-level 2 would require
   │     │  └─ __archspec 1 steamroller, which is missing on the system;
   │     ├─ _x86_64-microarch-level 2 would require
   │     │  └─ __archspec 1 westmere, which is missing on the system;
   │     ├─ _x86_64-microarch-level 2 would require
   │     │  └─ __archspec 1 x86_64_v2, which is missing on the system;
   │     ├─ _x86_64-microarch-level 3 would require
   │     │  └─ __archspec 1 broadwell, which is missing on the system;
   │     ├─ _x86_64-microarch-level 3 would require
   │     │  └─ __archspec 1 cannonlake, which is missing on the system;
   │     ├─ _x86_64-microarch-level 3 would require
   │     │  └─ __archspec 1 excavator, which is missing on the system;
   │     ├─ _x86_64-microarch-level 3 would require
   │     │  └─ __archspec 1 haswell, which is missing on the system;
   │     ├─ _x86_64-microarch-level 3 would require
   │     │  └─ __archspec 1 mic_knl, which is missing on the system;
   │     ├─ _x86_64-microarch-level 3 would require
   │     │  └─ __archspec 1 skylake, which is missing on the system;
   │     ├─ _x86_64-microarch-level 3 would require
   │     │  └─ __archspec 1 x86_64_v3, which is missing on the system;
   │     ├─ _x86_64-microarch-level 3 would require
   │     │  └─ __archspec 1 zen, which is missing on the system;
   │     ├─ _x86_64-microarch-level 3 would require
   │     │  └─ __archspec 1 zen2, which is missing on the system;
   │     ├─ _x86_64-microarch-level 3 would require
   │     │  └─ __archspec 1 zen3, which is missing on the system;
   │     ├─ _x86_64-microarch-level 4 would require
   │     │  └─ __archspec 1 cascadelake, which is missing on the system;
   │     ├─ _x86_64-microarch-level 4 would require
   │     │  └─ __archspec 1 icelake, which is missing on the system;
   │     ├─ _x86_64-microarch-level 4 would require
   │     │  └─ __archspec 1 skylake_avx512, which is missing on the system;
   │     ├─ _x86_64-microarch-level 4 would require
   │     │  └─ __archspec 1 x86_64_v4, which is missing on the system;
   │     ├─ _x86_64-microarch-level 4 would require
   │     │  └─ __archspec 1 zen4, which is missing on the system;
   │     └─ _x86_64-microarch-level 4 would require
   │        └─ __archspec 1 sapphirerapids, which is missing on the system;
   └─ awkward-cpp 38 would require
      └─ _x86_64-microarch-level >=3 , which cannot be installed (as previously explained)

So until mamba makes a new release with the patch mamba-org/mamba#3404 use of microarch levels is broken across all of conda-forge. :(

I'll make a PR to https://github.com/conda-forge/awkward-cpp-feedstock/ to revert conda-forge/awkward-cpp-feedstock#46 to fix this until a new mamba is released.

@matthewfeickert
Copy link
Member

Once PR conda-forge/awkward-cpp-feedstock#48 is merged and awkward-cpp is rebuilt this can get rerun and things should pass then.

@matthewfeickert
Copy link
Member

@conda-forge-admin, please rerender

@conda-forge-webservices
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

Copy link
Contributor

Hi! This is the friendly automated conda-forge-webservice.

I tried to rerender for you, but it looks like there was nothing to do.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/awkward-feedstock/actions/runs/10854884630.

* awkward 2.6.8 has requirement awkward-cpp==38.
@matthewfeickert
Copy link
Member

matthewfeickert commented Sep 13, 2024

Ah, so conda-forge/awkward-cpp-feedstock#48 was actually uncessary. The examples I gave with mamba failing to find the microarch level distributions were correct, except

  • conda-forge using mamba as the default solver for conda-build.

is apparently not right. The real problem was that

awkward 2.6.8 has requirement awkward-cpp==38

and this was still using

- awkward-cpp ==37

So conda-forge/awkward-cpp-feedstock#48 can be reverted by conda-forge/awkward-cpp-feedstock#49.

@matthewfeickert matthewfeickert merged commit 04ae177 into conda-forge:main Sep 13, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants