From bd001196d0480a9ed3f45956d5b3da7ae9a5637a Mon Sep 17 00:00:00 2001 From: Timothy Knight Date: Thu, 4 Sep 2014 23:25:39 -0700 Subject: CameraITS: Added some TODOs to remove hard-coded capture settings Change-Id: I47d3570cd87141eea34c4d61a68c2d7c04cc0b5d --- apps/CameraITS/pymodules/its/device.py | 2 +- apps/CameraITS/tests/scene0/test_capture_result_dump.py | 1 + apps/CameraITS/tests/scene0/test_metadata.py | 1 + apps/CameraITS/tests/scene0/test_param_sensitivity_burst.py | 1 + 4 files changed, 4 insertions(+), 1 deletion(-) (limited to 'apps') diff --git a/apps/CameraITS/pymodules/its/device.py b/apps/CameraITS/pymodules/its/device.py index cbf579e..ea763ef 100644 --- a/apps/CameraITS/pymodules/its/device.py +++ b/apps/CameraITS/pymodules/its/device.py @@ -291,7 +291,7 @@ class ItsSession(object): format(s) of the captured image. The formats may be "yuv", "jpeg", "dng", "raw", or "raw10". The default is a YUV420 frame ("yuv") corresponding to a full sensor frame. - + Note that one or more surfaces can be specified, allowing a capture to request images back in multiple formats (e.g.) raw+yuv, raw+jpeg, yuv+jpeg, raw+yuv+jpeg. If the size is omitted for a surface, the diff --git a/apps/CameraITS/tests/scene0/test_capture_result_dump.py b/apps/CameraITS/tests/scene0/test_capture_result_dump.py index ceac43a..40c3e45 100644 --- a/apps/CameraITS/tests/scene0/test_capture_result_dump.py +++ b/apps/CameraITS/tests/scene0/test_capture_result_dump.py @@ -27,6 +27,7 @@ def main(): with its.device.ItsSession() as cam: # Arbitrary capture request exposure values; image content is not # important for this test, only the metadata. + # TODO: Update test to ensure manual settings are within legal ranges. req = its.objects.manual_capture_request(100, 10*1000*1000) req["android.statistics.lensShadingMapMode"] = 1 cap = cam.do_capture(req, cam.CAP_YUV) diff --git a/apps/CameraITS/tests/scene0/test_metadata.py b/apps/CameraITS/tests/scene0/test_metadata.py index ff39290..861c071 100644 --- a/apps/CameraITS/tests/scene0/test_metadata.py +++ b/apps/CameraITS/tests/scene0/test_metadata.py @@ -31,6 +31,7 @@ def main(): # Arbitrary capture request exposure values; image content is not # important for this test, only the metadata. props = cam.get_camera_properties() + # TODO: Update test to ensure manual settings are within legal ranges. req = its.objects.manual_capture_request(100, 10*1000*1000) cap = cam.do_capture(req, cam.CAP_YUV) md = cap["metadata"] diff --git a/apps/CameraITS/tests/scene0/test_param_sensitivity_burst.py b/apps/CameraITS/tests/scene0/test_param_sensitivity_burst.py index ad94446..f906287 100644 --- a/apps/CameraITS/tests/scene0/test_param_sensitivity_burst.py +++ b/apps/CameraITS/tests/scene0/test_param_sensitivity_burst.py @@ -34,6 +34,7 @@ def main(): sens_range = props['android.sensor.info.sensitivityRange'] sens_step = (sens_range[1] - sens_range[0]) / NUM_STEPS sensitivities = range(sens_range[0], sens_range[1], sens_step) + # TODO: Update test to ensure manual settings are within legal ranges. reqs = [its.objects.manual_capture_request(s,10*1000*1000) for s in sensitivities] -- cgit v1.2.3