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

issue about get_denorm and ego2sensor #53

Open
WjzZwd opened this issue Oct 15, 2024 · 7 comments
Open

issue about get_denorm and ego2sensor #53

WjzZwd opened this issue Oct 15, 2024 · 7 comments

Comments

@WjzZwd
Copy link

WjzZwd commented Oct 15, 2024

image
image

I have a question about the get_reference_height function?

def get_reference_height(denorm):

ref_height = np.abs(denorm[3])/np.sqrt(denorm[0]**2 + denorm[1]**2 + denorm[2]**2)
This function represents the distance from the origin to the plane According to this schematic from BEVHeight , It's the distance from the origin of the blue(sensor) coordinate system to the xy plane of the green(ego) coordinate system so denorm represents the xy plane of the green (ego)coordinate system
BUT!!! in get_denorm
def get_denorm(sweepego2sweepsensor):
if you think of
ground_points_lidar = np. array ([[0.0, 0.0, 0.0], [0.0, 1.0, 0.0], [1.0, 1.0, 0.0]])
represents the xy plane of the bule( sensor )coordinate system
ground_points_cam = np.matmul(sweepego2sweepsensor, ground_points_lidar.T).T
then can be understood as the xy plane of the green (ego) coordinate system.
So!!! i wonder whether
np.matmul(sweepego2sweepsensor, ground_points_lidar.T).T
need to be changed to
np.matmul(sweepsensor2sweepego, ground_points_lidar.T).T

@WjzZwd
Copy link
Author

WjzZwd commented Oct 15, 2024

looking forward to your reply ,thanks!!!! @yanglei18

@yanglei18
Copy link
Collaborator

yanglei18 commented Oct 15, 2024

  • step 1: ground_points_lidar represents some points in ego coordinate system.
  • step 2: ground_points_cam = np.matmul(sweepego2sweepsensor, ground_points_lidar.T).T represents the same points in camera coordinate system.
  • step 4: denorm is based on the ground_points_cam
  • step 5: ref_height is based on the denorm.

@WjzZwd
Copy link
Author

WjzZwd commented Oct 15, 2024

  • step 1: ground_points_lidar represents some points in ego coordinate system.
  • step 2: ground_points_cam = np.matmul(sweepego2sweepsensor, ground_points_lidar.T).T represents the same points in camera coordinate system.
  • step 4: denorm is based on the ground_points_cam
  • step 5: ref_height is based on the denorm.

thanks!for your reply ~ and i still get some doubt that

1.ego coordinate system corresponds to the green coordinate system in this picture, right?

2.based on the formula in picture
image

we can infer that the func

def get_reference_height(denorm):

aim to get the distace from the origin to the plane determined by denorm .
so
----1)who is the origin(the coordinates of the inf camera? or other)?
----2)and who is the plane (xy plane in the ego system? or xy plane in the sensor system? )

thanks for your reply~~~

@WjzZwd
Copy link
Author

WjzZwd commented Oct 16, 2024

my question is based on the reproduction in DAIR-V2X dataset @yanglei18 thanks you !~

@yanglei18
Copy link
Collaborator

  • All the origin and the plane used in def get_reference_height(denorm): is represented in camera coordinates system.
  • The origin corresponds to the origin of camera coordinates system.
  • The plane is the xy plane of ego coordinates system.
  • The ego coordinates system is the virtual LiDAR system in DAIR dataste.

@WjzZwd
Copy link
Author

WjzZwd commented Dec 29, 2024

您的回复
The ego coordinates system is the virtual LiDAR system in DAIR dataste.
说论文中ego坐标系的就是DAIR-V2X中的虚拟雷达坐标系
但是BEVHeight图中的ego坐标系的xy平面和地面是重合的 而DAIR-V2X中 路端虚拟雷达的中心和雷达中心一致,并不在地面
所以请问为什么您说ego坐标系和虚拟雷达坐标系一致呢?
@yanglei18 thanks for your reply~~~

@WjzZwd
Copy link
Author

WjzZwd commented Jan 5, 2025

waiting for your reply @yanglei18 thanks~💗

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants