Skip to content
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

Install from explicit dest_path instead of relative path. #1238

Merged

Conversation

rohinb2
Copy link
Contributor

@rohinb2 rohinb2 commented Sep 6, 2024

No description provided.

Copy link

sentry-io bot commented Sep 6, 2024

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: runhouse/resources/hardware/cluster.py

Function Unhandled Issue
_sync_runhouse_to_cluster ValueError: Error installing runhouse on cluster node <34.229.108.213> ...
Event Count: 1

Did you find this useful? React with a 👍 or 👎

@rohinb2 rohinb2 force-pushed the 09-05-Temporary_fix_for_pydantic_version_error_with_FastAPI branch from e21e8f5 to e2932d6 Compare September 9, 2024 18:42
@rohinb2 rohinb2 force-pushed the 09-06-Install_from_explicit_dest_path_instead_of_relative_path branch from bc55fd1 to e0f1735 Compare September 9, 2024 18:42
@jlewitt1 jlewitt1 changed the base branch from 09-05-Temporary_fix_for_pydantic_version_error_with_FastAPI to graphite-base/1238 September 9, 2024 19:20
@jlewitt1 jlewitt1 changed the base branch from graphite-base/1238 to main September 9, 2024 19:28
@jlewitt1 jlewitt1 force-pushed the 09-06-Install_from_explicit_dest_path_instead_of_relative_path branch from e0f1735 to ac58cbe Compare September 9, 2024 19:34
Copy link
Collaborator

jlewitt1 commented Sep 9, 2024

Merge activity

@jlewitt1 jlewitt1 merged commit 50a73ea into main Sep 9, 2024
13 checks passed
@jlewitt1 jlewitt1 deleted the 09-06-Install_from_explicit_dest_path_instead_of_relative_path branch September 9, 2024 19:35
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.

3 participants