Skip to content

Conversation

@nathan-contino
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented May 19, 2025

Deploy Preview for viam-docs ready!

Name Link
🔨 Latest commit b8979fe
🔍 Latest deploy log https://app.netlify.com/projects/viam-docs/deploys/682e8b056e49100008a90b6b
😎 Deploy Preview https://deploy-preview-4322--viam-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 53 (🟢 up 4 from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 92 (no change from production)
PWA: 70 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@viambot viambot added the safe to build This pull request is marked safe to build from a trusted zone label May 19, 2025
@viambot
Copy link
Member

viambot commented May 19, 2025

It looks like the following files may have been renamed. Please ensure you set all needed aliases:
rename docs/data-ai/reference/triggers-configuration.md => static/include/webhooks.md (50%)

Copy link
Contributor

@npentrel npentrel left a comment

Choose a reason for hiding this comment

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

Can you split these changes into two PRs? I'm essentially happy to merge the run inference part but I am confused by the updates to the scripts

- the binary data ID and organization of the data you want to run inference on
- the name, version, and organization of the model you want to use for inference

The [`viam infer`](/dev/tools/cli/#infer) CLI command runs inference in the cloud on a piece of data using the specified ML model:
Copy link
Contributor

Choose a reason for hiding this comment

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

and this is also possible from the sdk? that might be good to show but can be added separately

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unfortunately not! I originally thought there was a way but only the protos exist right now. That's also why I wound up messing with the SDK doc generation scripts (now reverted), because I was getting some typescript failures and I had to fix them to continue investigating.

Copy link
Contributor

@npentrel npentrel left a comment

Choose a reason for hiding this comment

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

Thanks for the changes - good to go!

@nathan-contino nathan-contino merged commit 80351a4 into viamrobotics:main May 22, 2025
12 checks passed
@github-actions
Copy link

🔎💬 Inkeep AI search and chat service is syncing content for source 'Viam Docs'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to build This pull request is marked safe to build from a trusted zone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants