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.

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

add tensorflowjs use-case guide #19544

Merged
merged 7 commits into from
Mar 14, 2024

Conversation

craig-osterhout
Copy link
Contributor

@craig-osterhout craig-osterhout commented Feb 29, 2024

Description

Added TensorFlow.js face detection app use-case guide based on https://github.com/harsh4870/TensorJS-Face-Detection

https://deploy-preview-19544--docsdocker.netlify.app/guides/use-case/tensorflowjs/

Related issues or tickets

ENGDOCS-1984

Reviews

  • Technical review
  • Editorial review

Copy link

netlify bot commented Feb 29, 2024

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 143f24b
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/65f0a39759b0410008b9814b
😎 Deploy Preview https://deploy-preview-19544--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@craig-osterhout craig-osterhout marked this pull request as ready for review February 29, 2024 22:26
content/guides/use-case/tensorflowjs.md Outdated Show resolved Hide resolved
content/guides/use-case/tensorflowjs.md Outdated Show resolved Hide resolved
content/guides/use-case/tensorflowjs.md Outdated Show resolved Hide resolved
content/guides/use-case/tensorflowjs.md Outdated Show resolved Hide resolved
content/guides/use-case/tensorflowjs.md Outdated Show resolved Hide resolved
content/guides/use-case/tensorflowjs.md Outdated Show resolved Hide resolved
@craig-osterhout craig-osterhout marked this pull request as draft March 5, 2024 22:53
@craig-osterhout craig-osterhout marked this pull request as ready for review March 7, 2024 21:29
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
@dvdksn
Copy link
Collaborator

dvdksn commented Mar 8, 2024

This seems more like a sample than a guide, or? There aren't any actionable steps in here other than building and running a container.

@craig-osterhout
Copy link
Contributor Author

This seems more like a sample than a guide, or? There aren't any actionable steps in here other than building and running a container.

We could throw in Compose Watch with a source code change, and a push to Hub. That'll introduce build, run, develop, and share.
What do you think? Or any other ideas?

@dvdksn
Copy link
Collaborator

dvdksn commented Mar 11, 2024

yeah let's do that

Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
dvdksn
dvdksn previously approved these changes Mar 12, 2024
content/guides/use-case/tensorflowjs.md Outdated Show resolved Hide resolved
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
@craig-osterhout craig-osterhout merged commit da43b4a into docker:main Mar 14, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants