-
Notifications
You must be signed in to change notification settings - Fork 11
Update PySquared to 25w40 #13
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
Conversation
|
Hey @nateinaction it looks like the CI is failing here because the 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! |
|
@Mikefly123 Fixed in ca122b0 |
|
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 |
|
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. |
ineskhou
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Summary
This update bumps the version of
pysquaredto the latest 25w39 release.It also removed the use of
sleep_helper(because it's not really doing anything useful tbh) from themain.pyloop. 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!