Skip to content

Commit 3ac999f

Browse files
sacpisbettinaheim1tnguyen
authored
PR #2683 Follow up (#2705)
Signed-off-by: Sachin Pisal <[email protected]> Signed-off-by: Thien Nguyen <[email protected]> Signed-off-by: Bettina Heim <[email protected]> Co-authored-by: Bettina Heim <[email protected]> Co-authored-by: Thien Nguyen <[email protected]>
1 parent 7c64da6 commit 3ac999f

32 files changed

+161
-194
lines changed

.github/workflows/ci.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -234,7 +234,7 @@ jobs:
234234
strategy:
235235
matrix:
236236
platform: [amd64, arm64]
237-
python_version: ['3.10', '3.12']
237+
python_version: ['3.12'] # FIXME: add '3.10' back
238238
cuda_version: ["11.8", "12.0"]
239239
fail-fast: false
240240
uses: ./.github/workflows/python_wheels.yml
@@ -255,7 +255,7 @@ jobs:
255255
uses: ./.github/workflows/python_metapackages.yml
256256
with:
257257
cudaq_version: ${{ needs.python_wheels.outputs.cudaq_version }}
258-
python_versions: "['3.10', '3.12']"
258+
python_versions: "['3.12']" # fixme: reenable 3.10
259259
cuda_versions: "['', '11.8', '12.0']"
260260
wheel_artifacts: 'pycudaq-*'
261261

.github/workflows/config/validation_config.json

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,7 @@
66
"operating_systems":
77
[
88
"ubuntu:22.04",
9-
"fedora:39",
10-
"opensuse/leap:15.5"
9+
"fedora:39"
1110
]
1211
},
1312
{
@@ -17,8 +16,7 @@
1716
"ubuntu:22.04",
1817
"debian:12",
1918
"fedora:38",
20-
"redhat/ubi9:9.2",
21-
"opensuse/leap:15.5"
19+
"redhat/ubi9:9.2"
2220
]
2321
},
2422
{
@@ -27,8 +25,7 @@
2725
[
2826
"ubuntu:24.04",
2927
"fedora:39",
30-
"redhat/ubi9:9.2",
31-
"opensuse/leap:15.6"
28+
"redhat/ubi9:9.2"
3229
]
3330
}
3431
],

.github/workflows/docker_images.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -719,7 +719,7 @@ jobs:
719719
# Unfortunately, we need to install cuquantum for docs generation to work properly,
720720
# since using mock imports in the autodocs configuration doesn't work properly.
721721
# See also https://github.com/sphinx-doc/sphinx/issues/11211.
722-
docker exec cuda-quantum-dev bash -c "python3 -m pip install cuquantum-python-cu12~=24.11"
722+
docker exec cuda-quantum-dev bash -c "python3 -m pip cache purge && python3 -m pip install cuquantum-python-cu12~=24.11"
723723
724724
(docker exec cuda-quantum-dev bash -c "export $docs_version && bash scripts/build_docs.sh" && built=true) || built=false
725725
if $built; then docker cp cuda-quantum-dev:"/usr/local/cudaq/docs/." docs; \
@@ -820,7 +820,7 @@ jobs:
820820
docker cp docs/notebook_validation.py cuda-quantum:"/home/cudaq/notebook_validation.py"
821821
# In containers without GPU support, UCX does not work properly since it is configured to work with GPU-support.
822822
# Hence, don't enforce UCX when running these tests.
823-
docker exec cuda-quantum bash -c "python3 -m pip install pandas scipy pandas seaborn 'h5py<3.11' contfrac"
823+
docker exec cuda-quantum bash -c "python3 -m pip cache purge && python3 -m pip install pandas scipy pandas seaborn 'h5py<3.11' contfrac"
824824
(docker exec cuda-quantum bash -c "unset OMPI_MCA_pml && set -o pipefail && bash validate_container.sh | tee /tmp/validation.out") && passed=true || passed=false
825825
docker cp cuda-quantum:"/tmp/validation.out" /tmp/validation.out
826826
docker stop cuda-quantum

.github/workflows/publishing.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -839,11 +839,10 @@ jobs:
839839
- name: Basic validation (GPU backends)
840840
shell: bash
841841
run: |
842-
# We don't test the 'dynamics' target in these tests
843842
backends_to_test=`\
844843
for file in $(ls $CUDA_QUANTUM_PATH/targets/*.yml); \
845844
do
846-
if [[ "$file" != *"dynamics"* ]] && [ -n "$(cat $file | grep "gpu-requirements")" ]; then \
845+
if [ -n "$(cat $file | grep "gpu-requirements")" ]; then \
847846
basename $file | cut -d "." -f 1; \
848847
elif [ -n "$(basename $file | grep mqpu)" ]; then \
849848
echo remote-mqpu; \

.github/workflows/python_wheels.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -188,6 +188,13 @@ jobs:
188188
matrix:
189189
os_image: ${{ fromJSON(needs.create_test_config.outputs.json).os_images }}
190190
pip_install_flags: ['', '--user']
191+
exclude:
192+
- os_image: ubuntu:22.04
193+
pip_install_flags: '--user'
194+
- os_image: ubuntu:24.04
195+
pip_install_flags: '--user'
196+
- os_image: fedora:39
197+
pip_install_flags: ''
191198
fail-fast: false
192199

193200
steps:

.github/workflows/test_in_devenv.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -222,8 +222,9 @@ jobs:
222222
shell: bash
223223
run: |
224224
cd $CUDAQ_REPO_ROOT
225+
pip cache purge
225226
pip install iqm_client==16.1 --user -vvv
226-
pip install . --user -vvv
227+
pip install . -vvv # FIXME: --user causes package hash mismatch on ubuntu...
227228
pyinstall_status=$?
228229
if [ ! $pyinstall_status -eq 0 ]; then
229230
echo "::error file=test_in_devenv.yml:: Pip install of CUDA Quantum failed with status $pyinstall_status."

docker/build/devdeps.Dockerfile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -145,12 +145,12 @@ COPY --from=prereqs /usr/local/aws "$AWS_INSTALL_PREFIX"
145145

146146
# Install additional dependencies required to build and test CUDA-Q.
147147
RUN apt-get update && apt-get install --no-install-recommends -y wget ca-certificates \
148-
&& wget https://github.com/Kitware/CMake/releases/download/v3.26.4/cmake-3.26.4-linux-$(uname -m).tar.gz \
149-
&& tar xf cmake-3.26.4* && mv cmake-3.26.4-linux-$(uname -m)/ /usr/local/cmake-3.26/ && rm -rf cmake-3.26.4* \
148+
&& wget https://github.com/Kitware/CMake/releases/download/v3.28.4/cmake-3.28.4-linux-$(uname -m).tar.gz \
149+
&& tar xf cmake-3.28.4* && mv cmake-3.28.4-linux-$(uname -m)/ /usr/local/cmake-3.28/ && rm -rf cmake-3.28.4* \
150150
# NOTE: removing ca-certificates also remove python3-pip.
151151
&& apt-get remove -y wget ca-certificates \
152152
&& apt-get autoremove -y --purge && apt-get clean && rm -rf /var/lib/apt/lists/*
153-
ENV PATH="${PATH}:/usr/local/cmake-3.26/bin"
153+
ENV PATH="${PATH}:/usr/local/cmake-3.28/bin"
154154
# We must use h5py<3.11 because 3.11 doesn't include aarch64 Linux wheels.
155155
# https://github.com/h5py/h5py/issues/2408
156156
RUN apt-get update && apt-get install -y --no-install-recommends \

docker/build/devdeps.manylinux.Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ RUN dnf install -y --nobest --setopt=install_weak_deps=False \
7878
&& cmake --build . --target install --config Release \
7979
&& cd / && rm -rf /pybind11-project
8080

81-
RUN curl -L https://github.com/Kitware/CMake/releases/download/v3.26.4/cmake-3.26.4-linux-$(uname -m).sh -o cmake-install.sh \
81+
RUN curl -L https://github.com/Kitware/CMake/releases/download/v3.28.4/cmake-3.28.4-linux-$(uname -m).sh -o cmake-install.sh \
8282
&& bash cmake-install.sh --skip-licence --exclude-subdir --prefix=/usr/local \
8383
&& rm cmake-install.sh
8484

docker/test/wheels/debian.Dockerfile

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ ENV VIRTUAL_ENV=/opt/venv
2424
RUN python${python_version} -m venv "$VIRTUAL_ENV"
2525
ENV PATH="$VIRTUAL_ENV/bin:$PATH"
2626
RUN if [ -n "$preinstalled_modules" ]; then \
27+
python${python_version} -m pip cache purge && \
2728
echo $preinstalled_modules | xargs python${python_version} -m pip install; \
2829
fi
2930

@@ -42,7 +43,8 @@ RUN if [ -n "$pip_install_flags" ]; then \
4243
sed -i 's/include-system-site-packages = false/include-system-site-packages = true/' $VIRTUAL_ENV/pyvenv.cfg; \
4344
fi
4445

45-
RUN python${python_version} -m pip install ${pip_install_flags} /tmp/$cuda_quantum_wheel
46+
RUN python${python_version} -m pip cache purge && \
47+
python${python_version} -m pip install ${pip_install_flags} /tmp/$cuda_quantum_wheel
4648
RUN if [ -n "$optional_dependencies" ]; then \
4749
cudaq_package=$(echo $cuda_quantum_wheel | cut -d '-' -f1 | tr _ -) && \
4850
python${python_version} -m pip install ${pip_install_flags} $cudaq_package[$optional_dependencies]; \

docker/test/wheels/fedora.Dockerfile

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ RUN dnf update -y --nobest expat \
2222
python$(echo $python_version | tr -d .) \
2323
&& python${python_version} -m ensurepip --upgrade
2424
RUN if [ -n "$preinstalled_modules" ]; then \
25+
python${python_version} -m pip cache purge && \
2526
echo $preinstalled_modules | xargs python${python_version} -m pip install; \
2627
fi
2728

@@ -35,7 +36,8 @@ COPY docs/sphinx/snippets/python /tmp/snippets/
3536
COPY python/tests /tmp/tests/
3637
COPY python/README*.md /tmp/
3738

38-
RUN python${python_version} -m pip install ${pip_install_flags} /tmp/$cuda_quantum_wheel
39+
RUN python${python_version} -m pip cache purge && \
40+
python${python_version} -m pip install ${pip_install_flags} /tmp/$cuda_quantum_wheel
3941
RUN if [ -n "$optional_dependencies" ]; then \
4042
cudaq_package=$(echo $cuda_quantum_wheel | cut -d '-' -f1 | tr _ -) && \
4143
python${python_version} -m pip install ${pip_install_flags} $cudaq_package[$optional_dependencies]; \

0 commit comments

Comments
 (0)