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

Remove _alt_options from from_config #1592

Draft
wants to merge 2 commits into
base: cluster-refactor
Choose a base branch
from

Conversation

carolineechen
Copy link
Collaborator

No description provided.

Copy link

sentry-io bot commented Dec 13, 2024

🔍 Existing Issues For Review

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

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

Function Unhandled Issue
ondemand_cluster NameError: name 'sky' is not defined main in ...
Event Count: 2
cluster ValueError: Resource flux-demo not found. _main...
Event Count: 1
📄 File: runhouse/resources/resource.py (Click to Expand)
Function Unhandled Issue
from_name ValueError: Resource flux-demo not found. _main...
Event Count: 1
---

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

Copy link
Collaborator Author

carolineechen commented Dec 13, 2024

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@carolineechen carolineechen force-pushed the cluster-refactor branch 5 times, most recently from 53330c2 to c70bd63 Compare December 13, 2024 23:21
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