Upgrade skypilot executor with 0.9.2#246
Conversation
hemildesai
left a comment
There was a problem hiding this comment.
Thank you so much for the contribution 🎉 . Other packagers should also be compatible with Skypilot now, will add support in a follow up PR.
Do the logs work fine with the upgrade? If not, we can follow up with a fix in a separate PR as well.
|
You might need to fix the formatting according to Ruff format / format (pull_request) for the CI check to pass. We can merge afterwards. |
Updates follow migration guidelines: https://docs.skypilot.co/en/latest/reference/faq.html\#migration-from-skypilot-0-8-1 Minor documentation improvement to ensure package compatibility during installation of sky Co-authored by: sahger.lad@thoughtworks.com rahul.punjabi@thoughtworks.com Signed-off-by: Meissane Chami <meissane.chami@thoughtworks.com> Signed-off-by: twmeissane <meissane.chami@thoughtworks.com>
8a3371a to
12bcd67
Compare
Hi @hemildesai the logs work fine on our end and they are available when we train on a single node. However when we move to multiple node we do encounter some issues both in v0.8 and v0.9 around the working directly /nemo_run/code. We believe this is potentially just come misconfiguration on our end. |
The working dir should be synced from local filesystem to |
Updates follow migration guidelines: https://docs.skypilot.co/en/latest/reference/faq.html\#migration-from-skypilot-0-8-1 Minor documentation improvement to ensure package compatibility during installation of sky
Co-authored by: sahger.lad@thoughtworks.com rahul.punjabi@thoughtworks.com