-
-
Notifications
You must be signed in to change notification settings - Fork 39
Add Modal quickstart guide images #118
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
base: main
Are you sure you want to change the base?
Conversation
|
👋 Hello @raimbekovm, thank you for submitting a -✅ Define a Purpose: Clearly explain the purpose of your fix or feature in your PR description, and link to any relevant issues. Ensure your commit messages are clear, concise, and adhere to the project's conventions. For more guidance, please refer to our Contributing Guide. Don't hesitate to leave a comment if you have any questions. Thank you for contributing to Ultralytics! 🚀 |
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔍 PR Review
Made with ❤️ by Ultralytics Actions
All 2 changed files are generated/vendored (skipped review)
📋 Skipped 2 files (lock files, generated, images, etc.)
docs/modal-dashboard-function-calls.pngdocs/modal-terminal-output.png
33dfa45 to
ba40083
Compare
ba40083 to
b40d841
Compare
|
Thanks for adding these Modal quickstart screenshots—please double-check they’re compressed to a reasonable size and don’t include any sensitive info; once CI is green we can proceed with review/merge (see the Documentation Workflow for our image/media guidelines). |
|
Thanks for the review! I've made the following changes:
The remaining screenshot is compressed and doesn't contain any sensitive information. |
|
Sounds good—please just make sure the Modal quickstart is referencing the new |
|
The Modal quickstart PR (ultralytics/ultralytics#23414) references the correct path: |
|
Thanks for confirming—I'll do a final quick check on the |
Add images for the Modal quickstart guide documentation:
modal-terminal-output.png- Terminal output showing YOLO26 inference resultsmodal-dashboard-function-calls.png- Modal dashboard showing successful function executionRelated to ultralytics/ultralytics#23414
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Adds new Modal quickstart guide images to the
ultralytics/assetsdocs set 🖼️📊 Key Changes
docs/modal-dashboard-function-calls.pngdocs/modal-terminal-output.png🎯 Purpose & Impact
📋 Skipped 2 files (lock files, generated, images, etc.)
docs/modal-dashboard-function-calls.pngdocs/modal-terminal-output.png