AccelerationConsortium / ac-microcourses

Microcourses hosted by the Acceleration Consortium for self-driving lab topics.
https://ac-microcourses.readthedocs.io/
MIT License
24 stars 3 forks source link

bug in as7341_sensor.py? #83

Closed programlich closed 1 month ago

programlich commented 1 month ago

The as7341_sensor.py in module 1.2 did not work out of the box. In fact I got an error for line 6, saying that the module machine._machine did not exist. Removing the ._machine extension did the trick. The working line looks like: from machine import I2C, Pin

Not sure, why this happens, but I definitely did not get the script running with the unmodified line.

sgbaird commented 1 month ago

I see. That looks like a bug on my part: https://github.com/ACC-HelloWorld/2-blink-and-read-programlich/blob/main/lib%2Fas7341_sensor.py#L6.

I think the file directly from self-driving-lab-demo will work (i.e., what gets uploaded during module 1), but I must have done something wonky when putting it in the assignment repo. Will fix.

Get Outlook for Androidhttps://aka.ms/AAb9ysg


From: programlich @.> Sent: Monday, October 7, 2024 3:36:22 PM To: AccelerationConsortium/ac-microcourses @.> Cc: Subscribed @.***> Subject: [AccelerationConsortium/ac-microcourses] bug in as7341_sensor.py? (Issue #83)

The as7341_sensor.py in module 1.2 did not work out of the box. In fact I got an error for line 6, saying that the module machine._machine did not exist. Removing the ._machine extension did the trick. The working line looks like: from machine import I2C, Pin

Not sure, why this happens, but I definitely did not get the script running with the unmodified line.

— Reply to this email directly, view it on GitHubhttps://github.com/AccelerationConsortium/ac-microcourses/issues/83, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AK25ABPC6I5XZJYYLGI3NODZ2LPDNAVCNFSM6AAAAABPQUNZ7CVHI2DSMVQWIX3LMV43ASLTON2WKOZSGU3TCMZQGA3TONA. You are receiving this because you are subscribed to this thread.Message ID: @.***>