Skip to content

Conversation

@crschardt
Copy link
Contributor

@crschardt crschardt commented Nov 30, 2025

Description

This PR switches image building from pguyot/arm-runner-action to PhotonVision/photon-image-runner. The new runner uses native arm runners on GitHub and is much faster than the emulation used by the pguyot version.

Images from a95914c tested and working on:

  • RaspberryPi4b
  • OrangePi5
  • OrangePi5pro
  • RubikPi3

Closes #2191

Meta

Merge checklist:

  • Pull Request title is short, imperative summary of proposed changes
  • The description documents the what and why
  • If this PR changes behavior or adds a feature, user documentation is updated
  • If this PR touches photon-serde, all messages have been regenerated and hashes have not changed unexpectedly
  • If this PR touches configuration, this is backwards compatible with settings back to v2024.3.1
  • If this PR touches pipeline settings or anything related to data exchange, the frontend typing is updated
  • If this PR addresses a bug, a regression test for it is added

@crschardt crschardt requested a review from samfreund December 1, 2025 05:16
@crschardt
Copy link
Contributor Author

Images from a95914c tested and working on:

  • RaspberryPi4b
  • OrangePi5
  • OrangePi5pro
  • RubikPi3

@samfreund
Copy link
Member

Wait till we bump the image-version.

@crschardt crschardt marked this pull request as ready for review December 7, 2025 14:09
@crschardt crschardt requested a review from a team as a code owner December 7, 2025 14:09
Copy link
Contributor

@mcm001 mcm001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add testing results to MR description but code looks good!

@crschardt crschardt enabled auto-merge (squash) December 7, 2025 17:52
@crschardt crschardt dismissed samfreund’s stale review December 7, 2025 18:30

Changes have been addressed

@samfreund samfreund force-pushed the build-images-with-photon-image-runner branch from ff24e91 to 927c0da Compare December 7, 2025 21:59
@crschardt crschardt merged commit 192e2a1 into PhotonVision:main Dec 7, 2025
41 checks passed
samfreund pushed a commit to samfreund/photonvision that referenced this pull request Dec 18, 2025
## Description

This PR switches image building from `pguyot/arm-runner-action` to
`PhotonVision/photon-image-runner`. The new runner uses native arm
runners on GitHub and is much faster than the emulation used by the
pguyot version.

Images from
PhotonVision@a95914c
tested and working on:

- [x] RaspberryPi4b
- [x] OrangePi5
- [x] OrangePi5pro
- [x] RubikPi3

Closes PhotonVision#2191

<!-- Any issues this pull request closes or pull requests this
supersedes should be linked with `Closes #issuenumber`. -->

## Meta

Merge checklist:
- [x] Pull Request title is [short, imperative
summary](https://cbea.ms/git-commit/) of proposed changes
- [x] The description documents the _what_ and _why_
- [ ] If this PR changes behavior or adds a feature, user documentation
is updated
- [ ] If this PR touches photon-serde, all messages have been
regenerated and hashes have not changed unexpectedly
- [ ] If this PR touches configuration, this is backwards compatible
with settings back to v2024.3.1
- [ ] If this PR touches pipeline settings or anything related to data
exchange, the frontend typing is updated
- [ ] If this PR addresses a bug, a regression test for it is added
spacey-sooty pushed a commit to spacey-sooty/photonvision that referenced this pull request Dec 22, 2025
## Description

This PR switches image building from `pguyot/arm-runner-action` to
`PhotonVision/photon-image-runner`. The new runner uses native arm
runners on GitHub and is much faster than the emulation used by the
pguyot version.

Images from
PhotonVision@a95914c
tested and working on:

- [x] RaspberryPi4b
- [x] OrangePi5
- [x] OrangePi5pro
- [x] RubikPi3

Closes PhotonVision#2191

<!-- Any issues this pull request closes or pull requests this
supersedes should be linked with `Closes #issuenumber`. -->

## Meta

Merge checklist:
- [x] Pull Request title is [short, imperative
summary](https://cbea.ms/git-commit/) of proposed changes
- [x] The description documents the _what_ and _why_
- [ ] If this PR changes behavior or adds a feature, user documentation
is updated
- [ ] If this PR touches photon-serde, all messages have been
regenerated and hashes have not changed unexpectedly
- [ ] If this PR touches configuration, this is backwards compatible
with settings back to v2024.3.1
- [ ] If this PR touches pipeline settings or anything related to data
exchange, the frontend typing is updated
- [ ] If this PR addresses a bug, a regression test for it is added
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