Feature/add option to force camera settings installation#11
Open
3DPrintingGeek wants to merge 10 commits intoGuilouz:mainfrom
Open
Feature/add option to force camera settings installation#113DPrintingGeek wants to merge 10 commits intoGuilouz:mainfrom
3DPrintingGeek wants to merge 10 commits intoGuilouz:mainfrom
Conversation
When the new revision of the AI cam is detected, the script shows an error, and prevents installing the macros. This makes sense if the AI cam is the only installed cam, but not if one has installed an additional USB cam, connected to the front USB port. The change gives an option, too continue the installation in that case, by asking if the user wants to continue.
The old script didn't allow to install the camera settings macros, when the new HW revision of the AI cam has been detected, even when the additional USB cam could benefit from being controlled by the macros. The changes made in this commit give a warning about the new HW revision not bein compatible but allow the user to acknowledge that he wants the camera settings macros installed nevertheless.
When using additional cameras there must be a way to select for which camera the subsequent macro calls of the camera settings macros should be executed. The default camera is the Creality AI camera. Usage: Before calling one of the other macros, make a call to CAM_SELECT cam_device=n with n being the number of the cameras video device. E.g.: If your second USB cam is /dev/video5 then call "CAM_SELECT cam_device=5". Every other camera macro call will now be for this camera until changed by another CAM_SELECT-call. Note: Not all camera macros will work with every camera, and some only if the camera is in a specific mode. It depends mostly on the cams hardware capabilities. Calling the CAM_SETTINGS macro will show the available settings and their current values.
…n' into base-with-othe
…ettings_installation
USB camera support is installed into $INITD/S50_usb_camera This change changes the wrong directory S99_usb_camera to S50_usb_camera
|
im going to try this with my stock nebula pad cam |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.