Skip to content
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

Add LED control to as7341 sensor wrapper class #257

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

Neil-YL
Copy link
Contributor

@Neil-YL Neil-YL commented Dec 6, 2024

Add LED control to as7341 sensor wrapper class

@Neil-YL
Copy link
Contributor Author

Neil-YL commented Dec 6, 2024

It seems both tests failed at the public demo test for sensor data timed out:

E _queue.Empty: Sensor data retrieval timed out (60 seconds)

.tox/default/lib/python3.10/site-packages/self_driving_lab_demo/utils/observe.py:108: Empty

Does the test rely on the actual light demo to run?

@sgbaird
Copy link
Member

sgbaird commented Dec 6, 2024

It seems both tests failed at the public demo test for sensor data timed out:

E _queue.Empty: Sensor data retrieval timed out (60 seconds)

.tox/default/lib/python3.10/site-packages/self_driving_lab_demo/utils/observe.py:108: Empty

Does the test rely on the actual light demo to run?

That's correct, though new microcontroller code still needs to be manually uploaded to the public test demo (currently in the office with the 3D printer). The tests failing is a known (and somewhat recent) issue, which I've been meaning to debug. It seems to be specific to the GitHub Actions unit tests, which makes it more slower and more difficult to troubleshoot.

@sgbaird sgbaird merged commit 59a2dd5 into sparks-baird:main Dec 6, 2024
3 of 5 checks passed
@Neil-YL Neil-YL deleted the my-feature branch December 6, 2024 21:17
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.

2 participants