Skip to content

Update checkout command to use release branch#15

Open
sagerkudrick wants to merge 1 commit into
AMD-Ecosystem:release/1.5.3b2from
sagerkudrick:patch-1
Open

Update checkout command to use release branch#15
sagerkudrick wants to merge 1 commit into
AMD-Ecosystem:release/1.5.3b2from
sagerkudrick:patch-1

Conversation

@sagerkudrick

@sagerkudrick sagerkudrick commented Apr 11, 2026

Copy link
Copy Markdown

install_dependencies.sh exists in the release/1.5.3b2 branch not main

Motivation

install_dependencies.sh doesn't exist in main, theirs has requirements.txt and requires the user to pip install -r requirements.txt rather than executing this script. ROCm install differs from the modern NVIDIA install

Furthermore, release/1.5.3b2's main/examples is 552 commits ahead of main

Technical Details

Test Plan

Test Result

./install_dependencies.sh fails because the file doesn't exist.
If you instead use pip install -r requirements.txt that main assumes, you get an incorrect installation as release/1.5.3b2 includes:
pip install git+https://github.com/rahul-goel/fused-ssim/@88169c51c22973ad8fd2429c3298d8356fdd5dc8 --no-build-isolation

Submission Checklist

install_dependencies.sh exists in the release/1.5.3b2 branch not main
@sagerkudrick

Copy link
Copy Markdown
Author

Just following up on this PR. The instructions differ between main and release/1.5.3b2, so I aligned the README with the release branch behavior.

Let me know if the intended target should instead be updated to match main, happy to adjust

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.

1 participant