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.

Sign up for GitHub

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

Jump to bottom

Optimizer should take num_nodes into account #28

Closed
concretevitamin opened this issue Nov 15, 2021 · 0 comments · Fixed by #277
Closed

Optimizer should take num_nodes into account #28

concretevitamin opened this issue Nov 15, 2021 · 0 comments · Fixed by #277
Assignees
@gmittal
Labels
P0

Comments

@concretevitamin
Copy link
Member

...e.g., during pricing calculation.
@gmittal gmittal self-assigned this Jan 26, 2022
@gmittal gmittal mentioned this issue Feb 8, 2022
Merged
JGoo1 pushed a commit to skypilot-sds/skypilot that referenced this issue Apr 28, 2023
@sukhoonjung @sds-github
Merge pull request skypilot-org#28 from PARLab/smoke_test_v1
af741ea 
Smoke test v1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gmittal gmittal

Labels
P0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Account for multiple nodes in optimizer cost skypilot-org/skypilot
2 participants
@concretevitamin @gmittal