Skip to content

Conversation

Mikefly123
Copy link
Member

Summary

This update bumps the version of pysquared to the latest 25w39 release.

It also removed the use of sleep_helper (because it's not really doing anything useful tbh) from the main.py loop. Instead we will be listening for commands during the time we would have originally been sleeping. This change increases the idle power consumption of the satellite, but makes it more responsive in doing so.

How was this tested

@ineskhou if you have a chance to test this quickly on a V5b that would be greatly appreciated!

  • Added new unit tests
  • Ran code on hardware (screenshots are helpful)
  • Other (Please describe)

@Mikefly123 Mikefly123 requested a review from ineskhou September 26, 2025 19:59
@Mikefly123 Mikefly123 self-assigned this Sep 26, 2025
@Mikefly123 Mikefly123 added enhancement New feature or request help wanted Extra attention is needed labels Sep 26, 2025
@Mikefly123
Copy link
Member Author

Hey @nateinaction it looks like the CI is failing here because the makefile is failing to bring in the pysquared folder from the upstream pysquared library.

I think we didn't actually update any of the implementation repos after our introduction of workspaces a few weeks ago. If you could give me a pointer here on what needs to be updated to reconcile this that would be great!

@nateinaction
Copy link
Member

@Mikefly123 Fixed in ca122b0

@Mikefly123
Copy link
Member Author

Thanks for the help on this @nateinaction! I will try and replicate this update to the other implementation repos.

It also looks like 25w36 had another breaking change where the Magnetometer Proto is not being detected correctly by Beacon. I've made an issue on the pysquared repo to track it:
proveskit/pysquared#310

@Mikefly123
Copy link
Member Author

Okay, for the time being I have just commented out the Magnetometer from Beacon to unblock this PR from going in. Hopefully we can get the upstream patch in by this week's Tuesday release.

@Mikefly123 Mikefly123 changed the title Update pysquared Update PySquared to 25w40 Oct 1, 2025
Copy link
Contributor

@ineskhou ineskhou left a comment

Choose a reason for hiding this comment

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

LGTM

@ineskhou ineskhou merged commit 2ef18a0 into main Oct 1, 2025
3 checks passed
@ineskhou ineskhou deleted the update-pysquared branch October 1, 2025 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants