-
Notifications
You must be signed in to change notification settings - Fork 13
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 optional Spectacular AI SDK to RAE build #28
base: humble
Are you sure you want to change the base?
Conversation
Dockerfile
Outdated
@@ -3,6 +3,8 @@ FROM luxonis/depthai-ros-rae AS builder | |||
ARG SIM=0 | |||
ARG CORE_NUM=1 | |||
ARG BUILD_TYPE="RelWithDebInfo" | |||
ARG INCLUDE_SPECTACULARAI_ROS=YES |
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 would set that to false and modify the workflow file to set that argument to true
.github/workflows/main.workflow.yaml
Outdated
@@ -46,6 +46,8 @@ jobs: | |||
ROS_DISTRO=${{ env.ROS_DISTRO }} | |||
USE_RVIZ=1 | |||
SIM=1 | |||
secrets: | | |||
"GITHUB_RAE_PAT_TOKEN=${{ secrets.GITHUB_RAE_PAT_TOKEN }}" |
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.
Needs to be changed to SPECTACULAR_AI_TOKEN
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.
- needs to be added to build and push sim step
Dockerfile
Outdated
&& git clone --single-branch --branch ${SPECTACULARAI_ROS_VERSION} https://github.com/SpectacularAI/ros.git sai_ros \ | ||
&& cd sai_ros \ | ||
&& apt-get -y install unzip --no-install-recommends \ | ||
&& ROS_DISTRO=$ROS_DISTRO DEPTHAI_WS=$UNDERLAY_WS GITHUB_RAE_PAT_TOKEN=$(cat /run/secrets/GITHUB_RAE_PAT_TOKEN) . ./scripts/download_and_build_static.sh \ |
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.
Change /run/secrets/GITHUB_RAE_PAT_TOKEN
to /run/secrets/SPECTACULAR_AI_TOKEN
@Bercon hi, are there any usage examples? I've seen a Spectacular AI demo in one of the previous Kickstarter updates. Looks cool. Any references would be greatly appreciated. Update: I just checked the repo. So it's for commercial usage only? |
Include Spetacular AI SDK in RAE Docker image