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 the ability to configure cameras #3

Open
curtiswest opened this issue Sep 12, 2017 · 0 comments
Open

Add the ability to configure cameras #3

curtiswest opened this issue Sep 12, 2017 · 0 comments

Comments

@curtiswest
Copy link
Owner

curtiswest commented Sep 12, 2017

The current implementation of PEPI does not support camera configuration.

If multiple cameras are used, it may be advantageous to be able to configure their parameters, e.g. white balance, exposure, ISO, etc. to achieve a consistent capture.

This enhancement proposes an extension of the CameraServer defined in pepi.thrift called ConfigurableCameraServer. This would take the form of:

service ConfigurableCameraServer extends CameraServer {
    i32 iso(1:i32 value),
    i32 white_balance_temp(1:i32 value),
    float shutter_speed(1:float value)
}

There will need to be some thoughtful consideration put into deciding which operations to include in this new extension. Additionally, will need to look into some mechanism whereby server access will fallback to a normal CameraServer if ConfigurableCameraServer is not implemented. Such a mechanism may already be available in Thrift?

The user interface will need to be updated to accept these configuration functions.

@curtiswest curtiswest changed the title Camera Configuration Add the ability to configure cameras Sep 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant