Skip to content

nfc: TRAB upgrade recorders at LCs 21,22,24,26,27 to EQR120s #2361

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

nfc: TRAB upgrade recorders at LCs 21,22,24,26,27 to EQR120s #2361

wants to merge 3 commits into from

Conversation

Thomas-Benson
Copy link
Contributor

@Thomas-Benson Thomas-Benson commented Feb 23, 2025 •
edited
Loading

Upgrading 5 CUSP-Mes to EQR120s:

  • Replacing CUSP-Me 3DE-20733 at LC 21 with EQR120 46493
  • Replacing CUSP-Me 3DE-20734 at LC 22 with EQR120 46506
  • Replacing CUSP-Me 3De20703 at LC 24 with EQR120 46522
  • Replacing CUSP-Me 3De20704 at LC 26 with EQR120 46528
  • Replacing CUSP-Me 3DE-20739 at LC 27 with EQR120 46530

TODO check orientation

@Thomas-Benson Thomas-Benson self-assigned this Feb 23, 2025
@Thomas-Benson Thomas-Benson requested review from ozym and a team as code owners February 23, 2025 22:32
@Thomas-Benson Thomas-Benson force-pushed the TRAB-Building-array-swap branch from 36cafe8 to 8ab918e Compare February 23, 2025 22:39
salichon
salichon approved these changes Mar 6, 2025
View reviewed changes
Copy link
Contributor

@salichon salichon left a comment •
edited
Loading

Choose a reason for hiding this comment

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

looks good to
new default orientation is 188 degrees ( shoudl be in relation to the building alignment too )

@salichon salichon self-requested a review March 6, 2025 02:20
@salichon
Copy link
Contributor

salichon commented Apr 2, 2025

Kia Ora @Thomas-Benson @CBurton90
Wassup with this TRAB PR ?
fallen into a crack o in woat of something to happen?
cheerios

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@salichon salichon salichon approved these changes

@CBurton90 CBurton90 Awaiting requested review from CBurton90

@gva6500 gva6500 Awaiting requested review from gva6500

@ozym ozym Awaiting requested review from ozym ozym is a code owner

Assignees

@Thomas-Benson Thomas-Benson

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Thomas-Benson @salichon