New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 613117 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Aug 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Feature



Sign in to add a comment

Factory: Implement robot arm fixture and a general IMU calibration process.

Project Member Reported by shunhsingou@chromium.org, May 19 2016

Issue description

We have some boards with IMU sensors that need to be calibrated during the manufacturing, and the calibration is done with automatic robot arm to move the device.

This issue tracks the changes made for the general robot fixture and the calibration flow for such requirements.
 

Comment 1 by hungte@chromium.org, May 19 2016

Please also make a design doc and put the link here, thanks.
A draft of the design doc can be found here: https://docs.google.com/document/d/1B-w4aOEuW9XSxPxZoYv-zffsuz9jDutSClaNmxWZOVY
Project Member

Comment 3 by bugdroid1@chromium.org, Jun 17 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/factory/+/fe1cd113ac94554b83a7a6aa5a634be965898d7c

commit fe1cd113ac94554b83a7a6aa5a634be965898d7c
Author: Shun-Hsing Ou <shunhsingou@chromium.org>
Date: Thu May 19 08:15:13 2016

Create a robot fixture framework

Now we have some boards that use a robot arm for calibration. It's
usually related to the IMU sensors. Create a framework for the robot
fixture and a general IMU test flow for such requirement.

BUG= chromium:613117 
TEST=make test

Change-Id: I569eb8369f0ad3ae1482cc0c232f450c6fddd983
Reviewed-on: https://chromium-review.googlesource.com/345753
Commit-Ready: Shun-Hsing Ou <shunhsingou@chromium.org>
Tested-by: Shun-Hsing Ou <shunhsingou@chromium.org>
Reviewed-by: Hung-Te Lin <hungte@chromium.org>

[add] https://crrev.com/fe1cd113ac94554b83a7a6aa5a634be965898d7c/py/test/pytests/robot_movement_unittest.py
[add] https://crrev.com/fe1cd113ac94554b83a7a6aa5a634be965898d7c/py/test/fixture/robot/factory_common.py
[add] https://crrev.com/fe1cd113ac94554b83a7a6aa5a634be965898d7c/py/test/fixture/utils.py
[add] https://crrev.com/fe1cd113ac94554b83a7a6aa5a634be965898d7c/py/test/fixture/robot/__init__.py
[add] https://crrev.com/fe1cd113ac94554b83a7a6aa5a634be965898d7c/py/test/fixture/robot/robot.py
[add] https://crrev.com/fe1cd113ac94554b83a7a6aa5a634be965898d7c/py/test/fixture/robot/six_dof_calibration_robot.py
[add] https://crrev.com/fe1cd113ac94554b83a7a6aa5a634be965898d7c/py/test/fixture/robot/dummy_robot.json
[add] https://crrev.com/fe1cd113ac94554b83a7a6aa5a634be965898d7c/py/test/pytests/robot_movement.py
[add] https://crrev.com/fe1cd113ac94554b83a7a6aa5a634be965898d7c/py/test/fixture/robot/algorithm.py
[add] https://crrev.com/fe1cd113ac94554b83a7a6aa5a634be965898d7c/py/test/fixture/robot/six_dof_calibration_robot_unittest.py

Project Member

Comment 4 by bugdroid1@chromium.org, Jun 22 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/factory/+/5c00ab8e07806a2f8000eea501ee3d9dd909e54d

commit 5c00ab8e07806a2f8000eea501ee3d9dd909e54d
Author: Shun-Hsing Ou <shunhsingou@chromium.org>
Date: Mon Jun 20 21:40:57 2016

robot: change the message order

We need to show preparing message before calling LoadDevice, so operator
can know the robot is under movement.

BUG= chromium:613117 
TEST=None

Change-Id: I667ec087bb7cd4f4c8a4cabc529d25886726098f
Reviewed-on: https://chromium-review.googlesource.com/354103
Commit-Ready: Shun-Hsing Ou <shunhsingou@chromium.org>
Tested-by: Shun-Hsing Ou <shunhsingou@chromium.org>
Reviewed-by: Hung-Te Lin <hungte@chromium.org>

[modify] https://crrev.com/5c00ab8e07806a2f8000eea501ee3d9dd909e54d/py/test/pytests/robot_movement.py

Comment 5 by hungte@chromium.org, Aug 11 2016

Hi Earl, do we have anything left or can we close this?
Status: Fixed (was: Started)
All changes already been merged and in used.

Sign in to add a comment