Skip to content

Notebook utils #9

Merged
yonromai merged 5 commits intomainfrom
romain/notebook-scripts
Jun 19, 2025
Merged

Notebook utils #9
yonromai merged 5 commits intomainfrom
romain/notebook-scripts

Conversation

@yonromai
Copy link
Copy Markdown
Contributor

No description provided.

@yonromai yonromai force-pushed the romain/notebook-scripts branch from 6b71110 to 2063943 Compare June 19, 2025 13:37
@yonromai yonromai requested a review from ravwojdyla June 19, 2025 13:39
Copy link
Copy Markdown
Contributor

@ravwojdyla ravwojdyla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

fi

# 3. Define repository details
REPO_URL="https://github.com/tabtab-labs/caduceus-distill.git"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New org

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(This still works with the auto-redirect, but) it's fixed in #10

print(f" Warm-up on {gpu_device} complete.")

except Exception as e:
print(f" An error occurred during warm-up on {gpu_device}: {e}")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Printing traceback here could make this more readable. You can see in #11. But in general this seems like a good use case for logger.exception 🤔

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great point, I'll add this tomorrow, when I debug the issue with NVidia driver. Thanks!

@yonromai yonromai merged commit 533805c into main Jun 19, 2025
2 checks passed
@yonromai yonromai deleted the romain/notebook-scripts branch June 19, 2025 19:02
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.

2 participants