Skip to content

Commit

Permalink
[RSDK-8859][RSDK-8386] Remove raspberry pi built-in model, test workf…
Browse files Browse the repository at this point in the history
…lows and ld flag for static library. (#4397)
  • Loading branch information
randhid authored Oct 11, 2024
1 parent 16647cf commit b3864fc
Show file tree
Hide file tree
Showing 25 changed files with 4 additions and 2,487 deletions.
23 changes: 1 addition & 22 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -181,25 +181,6 @@ jobs:
ghcr.io/viamrobotics/rdk-devenv:armhf-cache \
sudo -Hu testbot bash -lc 'cd /rdk && go test -v -tags=no_tflite ./...'
test_pi:
name: Test Raspberry Pi Code
runs-on: pi4
timeout-minutes: 5

steps:
- name: Clean workspace
run: |
shopt -s dotglob
sudo chown -R `whoami` ./
rm -rf ./*
- uses: actions/checkout@v3
with:
ref: ${{ github.event_name == 'pull_request_target' && github.event.pull_request.head.sha || github.ref }}

- name: Run go unit tests on Pi
run: make test-pi

motion_tests:
name: Test Longer-running Motion Plans if affected
uses: viamrobotics/rdk/.github/workflows/motion-tests.yml@main
Expand Down Expand Up @@ -257,20 +238,18 @@ jobs:

test_passing:
name: All Tests Passing
needs: [test_go, test_web_e2e, test_pi, test32, motion_tests]
needs: [test_go, test_web_e2e, test32, motion_tests]
runs-on: [ubuntu-latest]
if: always()
steps:
- name: Check Results
run: |
echo Go Unit Tests: ${{ needs.test_go.result }}
echo Go 32-bit Tests: ${{ needs.test32.result }}
echo Go Pi Tests: ${{ needs.test_pi.result }}
echo Web/E2E Tests: ${{ needs.test_web_e2e.result }}
echo Motion Tests: ${{ needs.motion_tests.result }}
[ "${{ needs.test_go.result }}" == "success" ] && \
[ "${{ needs.test32.result }}" == "success" ] && \
[ "${{ needs.test_pi.result }}" == "success" ] && \
[ "${{ needs.test_web_e2e.result }}" == "success" ] && \
[ "${{ needs.motion_tests.result }}" == "success" ]
Expand Down
5 changes: 0 additions & 5 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -93,11 +93,6 @@ test-go-no-race: tool-install
test-web:
npm run test:unit --prefix web/frontend

# test.short skips tests requiring external hardware (motors/servos)
test-pi:
go test -c -o $(BIN_OUTPUT_PATH)/test-pi go.viam.com/rdk/components/board/pi/impl
sudo $(BIN_OUTPUT_PATH)/test-pi -test.short -test.v

test-e2e:
go build $(LDFLAGS) -o bin/test-e2e/server web/cmd/server/main.go
./etc/e2e.sh -o 'run' $(E2E_ARGS)
Expand Down
2 changes: 1 addition & 1 deletion components/board/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import "go.viam.com/rdk/resource"
// SPIConfig enumerates a specific, shareable SPI bus.
type SPIConfig struct {
Name string `json:"name"`
BusSelect string `json:"bus_select"` // "0" or "1" for main/aux in libpigpio
BusSelect string `json:"bus_select"` // "0" or "1" are common on single board computers
}

// Validate ensures all parts of the config are valid.
Expand Down
9 changes: 0 additions & 9 deletions components/board/pi/common/common.go

This file was deleted.

33 changes: 0 additions & 33 deletions components/board/pi/common/config.go

This file was deleted.

166 changes: 0 additions & 166 deletions components/board/pi/common/errors.go

This file was deleted.

2 changes: 0 additions & 2 deletions components/board/pi/doc.go

This file was deleted.

Loading

0 comments on commit b3864fc

Please sign in to comment.