Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .github/workflows/amd_workflow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,8 @@ jobs:
# Apply mask to the extracted content
echo "::add-mask::$PAYLOAD"

# Compress and base64 encode the payload to match what the Python script expects
echo "$PAYLOAD" | python3 -c "import sys, zlib, base64; print(base64.b64encode(zlib.compress(sys.stdin.read().encode('utf-8'))).decode('ascii'))" > payload.json
# Now write to file (won't be logged since it's masked)
echo "$PAYLOAD" > payload.json
Comment on lines +42 to +43
Copy link

Copilot AI Aug 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing the compression and base64 encoding may break compatibility with Python scripts that expect the compressed format. Verify that the consuming Python scripts can handle raw JSON payloads or update them accordingly.

Suggested change
# Now write to file (won't be logged since it's masked)
echo "$PAYLOAD" > payload.json
# Compress and base64-encode the payload before writing to file
echo "$PAYLOAD" | gzip | base64 > payload.json

Copilot uses AI. Check for mistakes.

- name: Set venv directory based on runner
run: |
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/nvidia_workflow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,8 +46,8 @@ jobs:
# Apply mask to the extracted content
echo "::add-mask::$PAYLOAD"

# Compress and base64 encode the payload to match what the Python script expects
echo "$PAYLOAD" | python3 -c "import sys, zlib, base64; print(base64.b64encode(zlib.compress(sys.stdin.read().encode('utf-8'))).decode('ascii'))" > payload.json
# Now write to file (won't be logged since it's masked)
echo "$PAYLOAD" > payload.json
Comment on lines +49 to +50
Copy link

Copilot AI Aug 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing the compression and base64 encoding may break compatibility with Python scripts that expect the compressed format. Verify that the consuming Python scripts can handle raw JSON payloads or update them accordingly.

Copilot uses AI. Check for mistakes.

- name: Install uv
uses: astral-sh/setup-uv@v3
Expand Down
2 changes: 1 addition & 1 deletion scripts/github_test_payload.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,5 +4,5 @@
"main.py": "import torch\n\nprint(f\"CUDA available: {torch.cuda.is_available()}\")\nprint(f\"PyTorch version: {torch.__version__}\")\n\nif torch.cuda.is_available():\n x = torch.randn(5, device='cuda')\n print(f\"Random tensor on GPU: {x}\")"
},
"main": "main.py",
"mode": "test"
"mode": "script"
Copy link

Copilot AI Aug 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reverting the mode from 'test' to 'script' may affect how this payload is processed downstream. Ensure that all consumers of this payload file are compatible with the 'script' mode and that any test-specific logic is properly handled.

Suggested change
"mode": "script"
"mode": "test"

Copilot uses AI. Check for mistakes.
}
Loading