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 openpilot prefix to imports #29498

Merged
merged 12 commits into from
Aug 21, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
1 change: 1 addition & 0 deletions Dockerfile.openpilot
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ WORKDIR ${OPENPILOT_PATH}

COPY SConstruct ${OPENPILOT_PATH}

COPY ./openpilot ${OPENPILOT_PATH}/openpilot
COPY ./third_party ${OPENPILOT_PATH}/third_party
COPY ./site_scons ${OPENPILOT_PATH}/site_scons
COPY ./laika ${OPENPILOT_PATH}/laika
Expand Down
4 changes: 2 additions & 2 deletions common/api/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
import os
import requests
from datetime import datetime, timedelta
from common.basedir import PERSIST
from system.version import get_version
from openpilot.common.basedir import PERSIST
from openpilot.system.version import get_version

API_HOST = os.getenv('API_HOST', 'https://api.commadotai.com')

Expand Down
2 changes: 1 addition & 1 deletion common/basedir.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import os
from pathlib import Path

from system.hardware import PC
from openpilot.system.hardware import PC

BASEDIR = os.path.abspath(os.path.join(os.path.dirname(os.path.realpath(__file__)), "../"))

Expand Down
2 changes: 1 addition & 1 deletion common/kalman/simple_kalman.py
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
# pylint: skip-file
from common.kalman.simple_kalman_impl import KF1D as KF1D
from openpilot.common.kalman.simple_kalman_impl import KF1D as KF1D
assert KF1D
8 changes: 4 additions & 4 deletions common/kalman/tests/test_simple_kalman.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@
import timeit
import numpy as np

from common.kalman.simple_kalman import KF1D
from common.kalman.simple_kalman_old import KF1D as KF1D_old
from openpilot.common.kalman.simple_kalman import KF1D
from openpilot.common.kalman.simple_kalman_old import KF1D as KF1D_old


class TestSimpleKalman(unittest.TestCase):
Expand Down Expand Up @@ -54,8 +54,8 @@ def test_new_is_faster(self):
setup = """
import numpy as np

from common.kalman.simple_kalman import KF1D
from common.kalman.simple_kalman_old import KF1D as KF1D_old
from openpilot.common.kalman.simple_kalman import KF1D
from openpilot.common.kalman.simple_kalman_old import KF1D as KF1D_old

dt = 0.01
x0_0 = 0.0
Expand Down
3 changes: 2 additions & 1 deletion common/params.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
from common.params_pyx import Params, ParamKeyType, UnknownKeyName, put_nonblocking, put_bool_nonblocking # pylint: disable=no-name-in-module, import-error
from openpilot.common.params_pyx import Params, ParamKeyType, UnknownKeyName, put_nonblocking, \
put_bool_nonblocking # pylint: disable=no-name-in-module, import-error
assert Params
assert ParamKeyType
assert UnknownKeyName
Expand Down
2 changes: 1 addition & 1 deletion common/realtime.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

from setproctitle import getproctitle

from system.hardware import PC
from openpilot.system.hardware import PC


# time step for each process
Expand Down
2 changes: 1 addition & 1 deletion common/spinner.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import os
import subprocess
from common.basedir import BASEDIR
from openpilot.common.basedir import BASEDIR


class Spinner():
Expand Down
4 changes: 2 additions & 2 deletions common/tests/test_file_helpers.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
import unittest
from uuid import uuid4

from common.file_helpers import atomic_write_on_fs_tmp
from common.file_helpers import atomic_write_in_dir
from openpilot.common.file_helpers import atomic_write_on_fs_tmp
from openpilot.common.file_helpers import atomic_write_in_dir


class TestFileHelpers(unittest.TestCase):
Expand Down
2 changes: 1 addition & 1 deletion common/tests/test_numpy_fast.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import numpy as np
import unittest

from common.numpy_fast import interp
from openpilot.common.numpy_fast import interp


class InterpTest(unittest.TestCase):
Expand Down
2 changes: 1 addition & 1 deletion common/tests/test_params.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
import uuid
import unittest

from common.params import Params, ParamKeyType, UnknownKeyName, put_nonblocking, put_bool_nonblocking
from openpilot.common.params import Params, ParamKeyType, UnknownKeyName, put_nonblocking, put_bool_nonblocking

class TestParams(unittest.TestCase):
def setUp(self):
Expand Down
2 changes: 1 addition & 1 deletion common/text_window.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
import os
import time
import subprocess
from common.basedir import BASEDIR
from openpilot.common.basedir import BASEDIR


class TextWindow:
Expand Down
2 changes: 1 addition & 1 deletion common/transformations/camera.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import numpy as np

import common.transformations.orientation as orient
import openpilot.common.transformations.orientation as orient

## -- hardcoded hardware params --
eon_f_focal_length = 910.0
Expand Down
6 changes: 3 additions & 3 deletions common/transformations/coordinates.py
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
# pylint: skip-file
from common.transformations.orientation import numpy_wrap
from common.transformations.transformations import (ecef2geodetic_single,
from openpilot.common.transformations.orientation import numpy_wrap
from openpilot.common.transformations.transformations import (ecef2geodetic_single,
geodetic2ecef_single)
from common.transformations.transformations import LocalCoord as LocalCoord_single
from openpilot.common.transformations.transformations import LocalCoord as LocalCoord_single


class LocalCoord(LocalCoord_single):
Expand Down
10 changes: 5 additions & 5 deletions common/transformations/model.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import numpy as np

from common.transformations.camera import (FULL_FRAME_SIZE,
from openpilot.common.transformations.camera import (FULL_FRAME_SIZE,
get_view_frame_from_calib_frame)

# segnet
Expand Down Expand Up @@ -61,8 +61,8 @@ def get_segnet_frame_from_camera_frame(segnet_size=SEGNET_SIZE, full_frame_size=
### This function mimics the update_calibration logic in modeld.cc
### Manually verified to give similar results to xx.uncommon.utils.transform_img
def get_warp_matrix(rpy_calib, wide_cam=False, big_model=False, tici=True):
from common.transformations.orientation import rot_from_euler
from common.transformations.camera import view_frame_from_device_frame, eon_fcam_intrinsics, tici_ecam_intrinsics, tici_fcam_intrinsics
from openpilot.common.transformations.orientation import rot_from_euler
from openpilot.common.transformations.camera import view_frame_from_device_frame, eon_fcam_intrinsics, tici_ecam_intrinsics, tici_fcam_intrinsics

if tici and wide_cam:
intrinsics = tici_ecam_intrinsics
Expand All @@ -85,8 +85,8 @@ def get_warp_matrix(rpy_calib, wide_cam=False, big_model=False, tici=True):

### This is old, just for debugging
def get_warp_matrix_old(rpy_calib, wide_cam=False, big_model=False, tici=True):
from common.transformations.orientation import rot_from_euler
from common.transformations.camera import view_frame_from_device_frame, eon_fcam_intrinsics, tici_ecam_intrinsics, tici_fcam_intrinsics
from openpilot.common.transformations.orientation import rot_from_euler
from openpilot.common.transformations.camera import view_frame_from_device_frame, eon_fcam_intrinsics, tici_ecam_intrinsics, tici_fcam_intrinsics


def get_view_frame_from_road_frame(roll, pitch, yaw, height):
Expand Down
2 changes: 1 addition & 1 deletion common/transformations/orientation.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
import numpy as np
from typing import Callable

from common.transformations.transformations import (ecef_euler_from_ned_single,
from openpilot.common.transformations.transformations import (ecef_euler_from_ned_single,
euler2quat_single,
euler2rot_single,
ned_euler_from_ecef_single,
Expand Down
2 changes: 1 addition & 1 deletion common/transformations/tests/test_coordinates.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
import numpy as np
import unittest

import common.transformations.coordinates as coord
import openpilot.common.transformations.coordinates as coord

geodetic_positions = np.array([[37.7610403, -122.4778699, 115],
[27.4840915, -68.5867592, 2380],
Expand Down
2 changes: 1 addition & 1 deletion common/transformations/tests/test_orientation.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
import numpy as np
import unittest

from common.transformations.orientation import euler2quat, quat2euler, euler2rot, rot2euler, \
from openpilot.common.transformations.orientation import euler2quat, quat2euler, euler2rot, rot2euler, \
rot2quat, quat2rot, \
ned_euler_from_ecef

Expand Down
30 changes: 15 additions & 15 deletions common/transformations/transformations.pyx
Original file line number Diff line number Diff line change
@@ -1,20 +1,20 @@
# distutils: language = c++
# cython: language_level = 3
from common.transformations.transformations cimport Matrix3, Vector3, Quaternion
from common.transformations.transformations cimport ECEF, NED, Geodetic

from common.transformations.transformations cimport euler2quat as euler2quat_c
from common.transformations.transformations cimport quat2euler as quat2euler_c
from common.transformations.transformations cimport quat2rot as quat2rot_c
from common.transformations.transformations cimport rot2quat as rot2quat_c
from common.transformations.transformations cimport euler2rot as euler2rot_c
from common.transformations.transformations cimport rot2euler as rot2euler_c
from common.transformations.transformations cimport rot_matrix as rot_matrix_c
from common.transformations.transformations cimport ecef_euler_from_ned as ecef_euler_from_ned_c
from common.transformations.transformations cimport ned_euler_from_ecef as ned_euler_from_ecef_c
from common.transformations.transformations cimport geodetic2ecef as geodetic2ecef_c
from common.transformations.transformations cimport ecef2geodetic as ecef2geodetic_c
from common.transformations.transformations cimport LocalCoord_c
from openpilot.common.transformations.transformations cimport Matrix3, Vector3, Quaternion
from openpilot.common.transformations.transformations cimport ECEF, NED, Geodetic

from openpilot.common.transformations.transformations cimport euler2quat as euler2quat_c
from openpilot.common.transformations.transformations cimport quat2euler as quat2euler_c
from openpilot.common.transformations.transformations cimport quat2rot as quat2rot_c
from openpilot.common.transformations.transformations cimport rot2quat as rot2quat_c
from openpilot.common.transformations.transformations cimport euler2rot as euler2rot_c
from openpilot.common.transformations.transformations cimport rot2euler as rot2euler_c
from openpilot.common.transformations.transformations cimport rot_matrix as rot_matrix_c
from openpilot.common.transformations.transformations cimport ecef_euler_from_ned as ecef_euler_from_ned_c
from openpilot.common.transformations.transformations cimport ned_euler_from_ecef as ned_euler_from_ecef_c
from openpilot.common.transformations.transformations cimport geodetic2ecef as geodetic2ecef_c
from openpilot.common.transformations.transformations cimport ecef2geodetic as ecef2geodetic_c
from openpilot.common.transformations.transformations cimport LocalCoord_c


import cython
Expand Down
4 changes: 2 additions & 2 deletions docs/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,8 @@
import sys
from os.path import exists

from common.basedir import BASEDIR
from system.version import get_version
from openpilot.common.basedir import BASEDIR
from openpilot.system.version import get_version

sys.path.insert(0, os.path.abspath('.'))
sys.path.insert(0, os.path.abspath('..'))
Expand Down
1 change: 1 addition & 0 deletions docs/docker/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ WORKDIR ${OPENPILOT_PATH}

COPY SConstruct ${OPENPILOT_PATH}

COPY ./openpilot ${OPENPILOT_PATH}/openpilot
COPY ./body ${OPENPILOT_PATH}/body
COPY ./third_party ${OPENPILOT_PATH}/third_party
COPY ./site_scons ${OPENPILOT_PATH}/site_scons
Expand Down
2 changes: 1 addition & 1 deletion laika_repo
3 changes: 3 additions & 0 deletions release/files_common
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,9 @@ docs/INTEGRATION.md
docs/LIMITATIONS.md
site_scons/site_tools/cython.py

openpilot/__init__.py
openpilot/**

common/.gitignore
common/__init__.py
common/conversions.py
Expand Down
2 changes: 1 addition & 1 deletion scripts/count_cars.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
from collections import Counter
from pprint import pprint

from selfdrive.car.docs import get_all_car_info
from openpilot.selfdrive.car.docs import get_all_car_info

if __name__ == "__main__":
cars = get_all_car_info()
Expand Down
2 changes: 1 addition & 1 deletion scripts/disable-powersave.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#!/usr/bin/env python3
from system.hardware import HARDWARE
from openpilot.system.hardware import HARDWARE

if __name__ == "__main__":
HARDWARE.set_power_save(False)
2 changes: 1 addition & 1 deletion scripts/pyqt_demo.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
#!/usr/bin/env python3

from PyQt5.QtWidgets import QApplication, QLabel # pylint: disable=no-name-in-module, import-error
from selfdrive.ui.qt.python_helpers import set_main_window
from openpilot.selfdrive.ui.qt.python_helpers import set_main_window


if __name__ == "__main__":
Expand Down
4 changes: 2 additions & 2 deletions scripts/waste.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,10 @@
import time
import numpy as np
from multiprocessing import Process
from setproctitle import setproctitle # pylint: disable=no-name-in-module
from setproctitle import setproctitle

def waste(core):
os.sched_setaffinity(0, [core,]) # pylint: disable=no-member
os.sched_setaffinity(0, [core,])

m1 = np.zeros((200, 200)) + 0.8
m2 = np.zeros((200, 200)) + 1.2
Expand Down
24 changes: 12 additions & 12 deletions selfdrive/athena/athenad.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,17 +30,17 @@
import cereal.messaging as messaging
from cereal import log
from cereal.services import service_list
from common.api import Api
from common.basedir import PERSIST
from common.file_helpers import CallbackReader
from common.params import Params
from common.realtime import set_core_affinity
from system.hardware import HARDWARE, PC, AGNOS
from system.loggerd.config import ROOT
from system.loggerd.xattr_cache import getxattr, setxattr
from selfdrive.statsd import STATS_DIR
from system.swaglog import SWAGLOG_DIR, cloudlog
from system.version import get_commit, get_origin, get_short_branch, get_version
from openpilot.common.api import Api
from openpilot.common.basedir import PERSIST
from openpilot.common.file_helpers import CallbackReader
from openpilot.common.params import Params
from openpilot.common.realtime import set_core_affinity
from openpilot.system.hardware import HARDWARE, PC, AGNOS
from openpilot.system.loggerd.config import ROOT
from openpilot.system.loggerd.xattr_cache import getxattr, setxattr
from openpilot.selfdrive.statsd import STATS_DIR
from openpilot.system.swaglog import SWAGLOG_DIR, cloudlog
from openpilot.system.version import get_commit, get_origin, get_short_branch, get_version


# TODO: use socket constant when mypy recognizes this as a valid attribute
Expand Down Expand Up @@ -552,7 +552,7 @@ def getNetworks():

@dispatcher.add_method
def takeSnapshot() -> Optional[Union[str, Dict[str, str]]]:
from system.camerad.snapshot.snapshot import jpeg_write, snapshot
from openpilot.system.camerad.snapshot.snapshot import jpeg_write, snapshot
ret = snapshot()
if ret is not None:
def b64jpeg(x):
Expand Down
8 changes: 4 additions & 4 deletions selfdrive/athena/manage_athenad.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,10 @@
import time
from multiprocessing import Process

from common.params import Params
from selfdrive.manager.process import launcher
from system.swaglog import cloudlog
from system.version import get_version, is_dirty
from openpilot.common.params import Params
from openpilot.selfdrive.manager.process import launcher
from openpilot.system.swaglog import cloudlog
from openpilot.system.version import get_version, is_dirty

ATHENA_MGR_PID_PARAM = "AthenadPid"

Expand Down
14 changes: 7 additions & 7 deletions selfdrive/athena/registration.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,13 @@
from typing import Optional

from datetime import datetime, timedelta
from common.api import api_get
from common.params import Params
from common.spinner import Spinner
from common.basedir import PERSIST
from selfdrive.controls.lib.alertmanager import set_offroad_alert
from system.hardware import HARDWARE, PC
from system.swaglog import cloudlog
from openpilot.common.api import api_get
from openpilot.common.params import Params
from openpilot.common.spinner import Spinner
from openpilot.common.basedir import PERSIST
from openpilot.selfdrive.controls.lib.alertmanager import set_offroad_alert
from openpilot.system.hardware import HARDWARE, PC
from openpilot.system.swaglog import cloudlog


UNREGISTERED_DONGLE_ID = "UnregisteredDevice"
Expand Down
2 changes: 1 addition & 1 deletion selfdrive/athena/tests/helpers.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
from functools import wraps
from multiprocessing import Process

from common.timeout import Timeout
from openpilot.common.timeout import Timeout


class MockResponse:
Expand Down
Loading