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

A question about prune_3d_lane_by_visibility #18

Closed
Aguin opened this issue Jul 4, 2022 · 3 comments
Closed

A question about prune_3d_lane_by_visibility #18

Aguin opened this issue Jul 4, 2022 · 3 comments
Assignees
Labels
bug Something isn't working documentation Improvements or additions to documentation

Comments

@Aguin
Copy link

Aguin commented Jul 4, 2022

Hi @zihanding819 ,
I notice that gt_lanes = [prune_3d_lane_by_visibility(np.array(gt_lane), np.array(gt_visibility[k])) for k, gt_lane enumerate(gt_lanes)] is removed in v1.1 and gt_visibility is not used in bench, does this mean we should not use the visibility labels in OpenLane?

@zihanding819
Copy link
Contributor

Hi @Aguin ,
In fact, in v1.1 visibility is still taken into account when calculating cost and error, instead of removing all invisible points in gt at the beginning. When calculating the cost, we divide it into 3 categories by the visibility relationship between pred and gt. And when calculating x_erorr / z_error, only points that are visible in both pred and gt are considered.

@Aguin
Copy link
Author

Aguin commented Jul 5, 2022

@zihanding819 thanks for your reply, gt_visibility_mat is generated by min_y and max_y here https://github.com/OpenPerceptionX/OpenLane/blob/main/eval/LANE_evaluation/lane3d/eval_3D_lane.py#L121, note that invisible points are kept when calculating min_y and max_y.

@Aguin Aguin mentioned this issue Jul 7, 2022
@ChonghaoSima ChonghaoSima added bug Something isn't working documentation Improvements or additions to documentation labels Sep 13, 2022
@ChonghaoSima ChonghaoSima reopened this Sep 13, 2022
@zihanding819
Copy link
Contributor

It is indeed necessary to prune invisible gt points before evaluation and we have fixed this in the new version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

4 participants