Skip to content
This repository was archived by the owner on Sep 10, 2025. It is now read-only.

Conversation

@Gasoonjia
Copy link
Contributor

@Gasoonjia Gasoonjia commented Jul 9, 2024

Current install_requirements.sh statically uses python3 to check python version; however, python3 may not exist in user's env and we may not use python3 during installation. Change to use $PYTHON_EXECUTABLE instead.

Current install_requirements.sh statically uses python3 to check python version; however, python3 may not exist in user's env. Change to use $PYTHON_EXECUTABLE
@Gasoonjia Gasoonjia requested a review from Jack-Khuu July 9, 2024 00:12
@pytorch-bot
Copy link

pytorch-bot bot commented Jul 9, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/torchchat/884

Note: Links to docs will display an error until the docs builds have been completed.

❌ 1 New Failure, 1 Cancelled Job

As of commit c421cd2 with merge base b6b6c1e (image):

NEW FAILURE - The following job has failed:

CANCELLED JOB - The following job was cancelled. Please retry:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jul 9, 2024
@Jack-Khuu
Copy link
Contributor

Manually merging given pre-existing test failure

Attempting fix in #887

@Jack-Khuu Jack-Khuu merged commit c7e6d16 into main Jul 9, 2024
fduwjj pushed a commit that referenced this pull request Jul 11, 2024
Current install_requirements.sh statically uses python3 to check python version; however, python3 may not exist in user's env. Change to use $PYTHON_EXECUTABLE
malfet pushed a commit that referenced this pull request Jul 17, 2024
Current install_requirements.sh statically uses python3 to check python version; however, python3 may not exist in user's env. Change to use $PYTHON_EXECUTABLE
malfet pushed a commit that referenced this pull request Jul 17, 2024
Current install_requirements.sh statically uses python3 to check python version; however, python3 may not exist in user's env. Change to use $PYTHON_EXECUTABLE
malfet pushed a commit that referenced this pull request Jul 17, 2024
Current install_requirements.sh statically uses python3 to check python version; however, python3 may not exist in user's env. Change to use $PYTHON_EXECUTABLE
malfet pushed a commit that referenced this pull request Jul 17, 2024
Current install_requirements.sh statically uses python3 to check python version; however, python3 may not exist in user's env. Change to use $PYTHON_EXECUTABLE
malfet pushed a commit that referenced this pull request Jul 17, 2024
Current install_requirements.sh statically uses python3 to check python version; however, python3 may not exist in user's env. Change to use $PYTHON_EXECUTABLE
malfet pushed a commit that referenced this pull request Jul 17, 2024
Current install_requirements.sh statically uses python3 to check python version; however, python3 may not exist in user's env. Change to use $PYTHON_EXECUTABLE
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants