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

On chip cleanup not functional in new Bookworm build #66

Open
Marcus8675 opened this issue Mar 10, 2024 · 2 comments
Open

On chip cleanup not functional in new Bookworm build #66

Marcus8675 opened this issue Mar 10, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@Marcus8675
Copy link

I was unable to disable the on chip cleanup using the new build on a Bookworm system. Changing the setting to false in the pilomar.json file had no effect. Upon boot on chip cleanup was listed as ENABLED. Using the camera menu to disable this feature also seems to have no effect. When camera information is displayed from this menu if always shows ENABLED. The main capture window also shows ENABLED.

@Short-bus Short-bus added the bug Something isn't working label Mar 10, 2024
@Short-bus Short-bus self-assigned this Mar 10, 2024
@Short-bus
Copy link
Owner

Hi. Well spotted, I'll need to clean up the UI a bit for that. Good to have a second pair of eyes running over it all.

Under Bookworm the camera is run by libcamera and that now has the 'denoise' option when you call it.
So the libcamera command templates (in the parameter file) have the '--denoise off ' option set in each case.
As a result the earlier onchip cleanup solution is only used with Buster builds.
So the onchip cleanup references are misleading under Bookworm because it's handled differently. I'll need to do some tidying up, I'd not noticed that inconsistency earlier, but it makes sense to hide that misleading info on a Bookworm install.

With this latest version I've moved more camera options into the command templates instead of having them hardcoded into the program. So hopefully overall it's easier to play with different camera settings as needed.

@Short-bus
Copy link
Owner

Testing modified logic.

In Buster:

The original DisableCleanup parameter and UI references work as before.

In Bookworm:

The DisableCleanup parameter will be removed.
The UI references now check the CameraLightCommand parameter to see if the '--denoise off' parameter is set.
If you try to enable/disable cleanup via the menu options it will recommend that you edit the command templates in the parameter file.
To enable on-chip cleanup, remove the '--denoise off' item from the command templates.
To disable on-chip cleanup, add the '--denoise off' item to the command templates.
The default templates have '--denoise off' in place. So you have to choose to turn on-chip cleanup back on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants