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

Need of gt_height and gt_pitch for inference: GeoNet3D_ext #16

Open
gautamjain1009 opened this issue Apr 7, 2022 · 4 comments
Open

Comments

@gautamjain1009
Copy link

Hey! I have explored your implementation quite nicely. The GeoNet3D_ext model takes as an input the features from the lane segmentation model. Thus those features are then fed to the GeoNet3D_ext model for 3d lane detection. From your implementation, I have realized that you are not predicting camera height and camera pitch instead you are using the gt camera height and gt camera pitch for the same and in your training script you are updating your M_inv using the same gt_camera_height and gt_camera_pitch.

if not args.fix_cam and not args.pred_cam:

            if not args.fix_cam and not args.pred_cam:
                model2.update_projection(args, gt_hcam, gt_pitch)

I tried to run the train script for GeoNet3D_ext and try to found out that gt_cam_height == pred_cam_height and the same for pitch.

Whereas in the unofficial implementation of 3D lanenet as per the train script, the height and pitch are different from gt's.

**Consider a real-world scenario where I want to use the pre-trained models provided by you for 3D lane Detection: ** in that particular case camera_height and camera_pitch are fixed as not really predicted by the model. Does in that particular case the results for 3d lane detection will differ? Have you tested your approach with any real-world data?

@CompliceU
Copy link

@gautamjain1009 Hi, have you tried training with OpenLane?

@gautamjain1009
Copy link
Author

WIP. I will train soon and will open source the code for that.

@CompliceU
Copy link

@gautamjain1009 That's great too, I'm currently trying this to work and am having a huge problem preventing it from training properly

@CompliceU
Copy link

@gautamjain1009 Hello, I don't know if you have tried it. I tried it. I feel that the effect is very different from the results provided by the author of openlane. During the training process, the loss of the verification part has not been effectively reduced.

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