-
Notifications
You must be signed in to change notification settings - Fork 0
Add support for new params and options #3
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
Conversation
| | `api_key` | YES | Your project number (see instruction below) | | ||
| | `impulse_id` | NO | Impulse ID. If this is unset then the default impulse is used | | ||
| | `engine` | NO | Inferencing engine type. See [docs for details](https://docs.edgeimpulse.com/apis/studio/jobs/build-on-device-model#body-engine) | | ||
| | `deployment_type` | NO | Type of the deployment, default is `zip` | |
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.
Do we have documented somewhere in our docs the different deployment_type ?
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 don't think so, as it is not a part of the API, but Studio code. I will request from Dev Rel team to provide a solution for listing all deployment types.
…in permissions Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
francovaro
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.
Just update the readme
francovaro
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.
Tested, works.
This PR adds several improvements:
Fixes edgeimpulse/edgeimpulse#13393