Skip to content

Update install_requirements.sh #884

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 9, 2024
Merged

Update install_requirements.sh #884

merged 1 commit into from
Jul 9, 2024

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
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
49 of 51 checks passed
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 join this conversation on GitHub. Already have an account? Sign in to comment
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.

4 participants