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 note about raspi cam options being ignored #690

Merged
merged 1 commit into from Oct 29, 2020

Conversation

pdfowler
Copy link
Contributor

@pdfowler pdfowler commented Oct 28, 2020

Myself, and others have run into the problem of setting options for camera_raspi_options only to have them be ignored. It appears that this is due to some pi cam modules being detected as USB cams and not pi cams as described in #674

Short of rewriting the camera detection code, this PR adds a note in the base octopi.txt to help people running into the issue.

The note reads:

NOTE: Newer raspi cam modules are reporting as usb devices causing these options to be ignored. Set `camera="raspi"` to avoid these 

Add note about problem with raspi cams reporting as usb and config options not being used.
@guysoft
Copy link
Owner

guysoft commented Oct 28, 2020

@foosel can you confirm before I possibly confuse a lot people?

@pdfowler
Copy link
Contributor Author

pdfowler commented Oct 29, 2020

Re-reading it, I think including a link to #674 would be useful. Maybe:

NOTE: Newer raspi cam modules are reporting as usb devices causing these options
to be ignored. See https://github.com/guysoft/OctoPi/issues/674 for more information. 
Set `camera="raspi"` to avoid these issues.

@foosel
Copy link
Collaborator

foosel commented Oct 29, 2020

Don't have the hardware to confirm in person but can confirm that we saw that on the forums and it's caused some confusion that this would help mitigate.

Better would be new detection code but I can't help there at the moment, no hardware and no time.

@guysoft
Copy link
Owner

guysoft commented Oct 29, 2020

No hardware here neither to test, merging.

@guysoft guysoft merged commit e2f38c7 into guysoft:devel Oct 29, 2020
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.

None yet

3 participants