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

fix driver for automatic pin factory #281

Merged
merged 19 commits into from
Jul 10, 2024
Merged

Conversation

ch3p4ll3
Copy link
Contributor

No description provided.

@coveralls
Copy link

Coverage Status

coverage: 100.0% (+0.2%) from 99.831%
when pulling 36a4e18 on ch3p4ll3:master
into fec1ea9 on borisbu:develop.

@ch3p4ll3
Copy link
Contributor Author

Hi, this should fix the issues of yesterday. I tried it on my Pi5 with Raspbian and it works.

Btw, it seems that gpiozero creates a cache(?) file, at the beginning I got this error: py.warnings - WARNING - /home/pi/OctoPrint/venv/lib/python3.11/site-packages/gpiozero-2.0.1-py3.11.egg/gpiozero/devices.py:300: PinFactoryFallback: Falling back from lgpio: [Errno 2] No such file or directory: '.lgd-nfy-3'.

I discovered that this is a permissions problem in the folder where octoprint is started from (being started with systemd I think some folder in the root). To fix this I had to add WorkingDirectory=<your octoprint folder>, eg: WorkingDirectory=/home/pi/OctoPrint to the file /etc/systemd/system/octoprint.service in the section Service. Then I reloaded the systemd services: sudo systemctl daemon-reload and finally restarted octoprint: sudo systemctl restart octoprint

This evening I'll try it on my Pi3 B+ with OctoPi. Let me know if you encounter some errors

Copy link
Collaborator

@RobinTail RobinTail left a comment

Choose a reason for hiding this comment

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

OK 👍🏽

@RobinTail RobinTail added the refactoring Better implementation of the same thing label Jul 10, 2024
@RobinTail RobinTail merged commit 197d511 into borisbu:develop Jul 10, 2024
16 checks passed
@RobinTail RobinTail mentioned this pull request Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Better implementation of the same thing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants