-
Notifications
You must be signed in to change notification settings - Fork 8
Add dc docs #210
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 dc docs #210
Conversation
|
Very nice 👯 and comprehensive. I will have to try it out before the workshop, and provide a better feedback. |
Yeah. I would say wait until we have release ready to try it out and that will bring together the whole flow. This PR is more to know from content perspective if everything you wish to see is there. I have the same confusion regarding links because it depends which is the landing point for users. |
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.
Copilot reviewed 6 out of 8 changed files in this pull request and generated no comments.
Files not reviewed (2)
- docs/psoc6/integrations/deepcraft_integration.rst: Language not supported
- docs/psoc6/quickref.rst: Language not supported
Signed-off-by: NikhitaR-IFX <nikhita.rajasekhar@infineon.com>
Signed-off-by: Eder Julian <Julian.Eder@infineon.com>
ederjc
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.
I did the changes as discussed and rebased to remove "fix" commits.
That is why we should not trust AI with everything 😅 |
|
|
||
| Example Projects | ||
| ================== | ||
| - `Edge AI-based baby cry detector with Home Assistant integration <https://www.hackster.io/Infineon_Team/ai-baby-cry-detector-with-home-assistant-integration-05576f>`_ |
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.
@ederjc : I thought of the article but this is not depending on the integration explained above. Should this still find a place here?
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.
As discussed, we keep it for now and later replce/update with right articles.
Summary
Added documentation explaining the DEEPCRAFT_MPY integration.
*Note: the placeholder images for last two points in model deployment will be done once I am able to get the images post python script working fine.
Here is the docs link: https://ifx-micropython.readthedocs.io/en/add-dc-docs/psoc6/integrations/deepcraft_integration.html
You can also help me know if we want to link articles or projects we do based on this integration under "Our published projects".
I see it can be too much to maintain and we can instead link only protip for each category which is one time? Let me know your views.