Skip to content

Commit 13d4277

Browse files
FloWuenneclaude
andcommitted
🐛 fix: resolve cleanup jq parsing and cellprofiler build issues
- Fix jq error with paginated API responses using jq -s 'add' - Pin numpy<2 and pre-install scikit-learn for cellprofiler Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 34db9b2 commit 13d4277

2 files changed

Lines changed: 6 additions & 5 deletions

File tree

.github/workflows/cleanup-pr-images.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,17 +22,17 @@ jobs:
2222
2323
echo "Cleaning up images for PR #${PR_NUMBER}..."
2424
25-
# Get all versions of the package
25+
# Get all versions of the package (merge paginated arrays)
2626
VERSIONS=$(gh api \
2727
-H "Accept: application/vnd.github+json" \
2828
-H "X-GitHub-Api-Version: 2022-11-28" \
2929
"/orgs/seqeralabs/packages/container/${PACKAGE_NAME}/versions" \
30-
--paginate 2>/dev/null || echo "[]")
30+
--paginate 2>/dev/null | jq -s 'add // []')
3131
3232
# Find all versions with tags ending in -pr<PR_NUMBER>
3333
echo "$VERSIONS" | jq -r --arg pr "pr${PR_NUMBER}" '
3434
.[] |
35-
select(.metadata.container.tags[] | endswith("-" + $pr)) |
35+
select(.metadata.container.tags[]? | endswith("-" + $pr)) |
3636
.id
3737
' | while read -r VERSION_ID; do
3838
if [[ -n "$VERSION_ID" ]]; then

kasmVNC/cellprofiler/Dockerfile

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,8 +42,9 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
4242
# Set JAVA_HOME for CellProfiler
4343
ENV JAVA_HOME=/usr/lib/jvm/default-java
4444

45-
# Install CellProfiler (requires numpy first)
46-
RUN pip3 install --no-cache-dir numpy && \
45+
# Install CellProfiler with compatible dependencies
46+
# Pin numpy<2 to avoid scikit-learn build issues with numpy 2.x
47+
RUN pip3 install --no-cache-dir "numpy<2" scikit-learn && \
4748
pip3 install --no-cache-dir cellprofiler
4849

4950
# Single-app mode: application launches directly, no desktop

0 commit comments

Comments
 (0)