-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
Format changes from running black on file
so that debugging is easier on rpi
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.
pls address comments about docstrings, and ensuring that hardware
is buried within the if __name__ == "__main__":
guard
client/drivers/data_structures.py
Outdated
Make a valid Picture. | ||
Args: | ||
message: Message to send to the user | ||
message_time: Time to sleep for after displaying message. |
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.
Should clarify that this is in units of SECONDS (other code is in units of milliseconds)
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.
agreed, done
return_me.matched = True | ||
return_me.user_id = user_id | ||
return return_me | ||
def _clear_buttons(self) -> None: |
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.
docstring pls
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.
done
client/drivers/main.py
Outdated
@@ -575,6 +398,7 @@ def handle_sniff_feedback(auspost: ControlledData) -> bool: | |||
|
|||
|
|||
## LAUNCH | |||
hardware = initialise_hardware() |
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.
This should be inside the if __name__ == "__main__":
guard, or buried somewhere within.
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.
done
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
Most changes in
data_structures
are from black formatting on save. There are two new methods for a simpler message display and waiting for a button press at the bottom ofHardwareComponents
.Picture
andFace
class have been deleted as they are no longer used anywhere.