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 optional Spectacular AI SDK to RAE build #28

Open
wants to merge 2 commits into
base: humble
Choose a base branch
from

Conversation

Bercon
Copy link

@Bercon Bercon commented Sep 19, 2023

Include Spetacular AI SDK in RAE Docker image

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
Copy link
Collaborator

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

@@ -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 }}"
Copy link
Collaborator

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

Copy link
Collaborator

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 \
Copy link
Collaborator

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 Bercon requested a review from Serafadam October 4, 2023 07:47
@sskorol
Copy link
Contributor

sskorol commented Oct 17, 2023

@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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants