"packaging/git@developer.sourcefind.cn:OpenDAS/vision.git" did not exist on "0b009ae3868610a6021ef1445790419c6a1fbf34"
Commit 8a96770d authored by Jiali Duan's avatar Jiali Duan Committed by Facebook GitHub Bot
Browse files

Update the docstring for try_get_projection_transform

Summary: Update the docstring for try_get_projection_transform on the API design.

Reviewed By: kjchalup

Differential Revision: D39227333

fbshipit-source-id: c9d0e625735d4972116d1f71865fb9b763e684de
parent 84fa9666
...@@ -6,7 +6,7 @@ ...@@ -6,7 +6,7 @@
import math import math
import warnings import warnings
from typing import List, Optional, Sequence, Tuple, Union from typing import Any, Dict, List, Optional, Sequence, Tuple, Union
import numpy as np import numpy as np
import torch import torch
...@@ -1851,12 +1851,15 @@ def get_screen_to_ndc_transform( ...@@ -1851,12 +1851,15 @@ def get_screen_to_ndc_transform(
return transform return transform
def try_get_projection_transform(cameras: CamerasBase, kwargs) -> Optional[Transform3d]: def try_get_projection_transform(
cameras: CamerasBase, cameras_kwargs: Dict[str, Any]
) -> Optional[Transform3d]:
""" """
Try block to get projection transform. Try block to get projection transform from cameras and cameras_kwargs.
Args: Args:
cameras instance, can be linear cameras or nonliear cameras cameras: cameras instance, can be linear cameras or nonliear cameras
cameras_kwargs: camera parameters to be passed to cameras
Returns: Returns:
If the camera implemented projection_transform, return the If the camera implemented projection_transform, return the
...@@ -1865,7 +1868,7 @@ def try_get_projection_transform(cameras: CamerasBase, kwargs) -> Optional[Trans ...@@ -1865,7 +1868,7 @@ def try_get_projection_transform(cameras: CamerasBase, kwargs) -> Optional[Trans
transform = None transform = None
try: try:
transform = cameras.get_projection_transform(**kwargs) transform = cameras.get_projection_transform(**cameras_kwargs)
except NotImplementedError: except NotImplementedError:
pass pass
return transform return transform
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment