-
Notifications
You must be signed in to change notification settings - Fork 373
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
factory test depth #403
base: depth-testing
Are you sure you want to change the base?
factory test depth #403
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you address these ?
4b434d6
to
76c73d9
Compare
depth_test/depthtest.py
Outdated
gt_plane = (gt_normal, -self.true_distance) | ||
gt_plane_dist = roi_point.dot(gt_plane[0]) + gt_plane[1] | ||
plane_offset_error += fitPlaneDist | ||
gt_offset_error += gt_plane_dist |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change this to Absolute error. Naming adjustment.
depth_test/depthtest.py
Outdated
elif self.ui.r_center.isChecked(): | ||
side = 'center' | ||
file.write(f'{int(time.time())},{mx_id},{self.true_distance},{self.z_distance},{self.plane_fit_mse},\ | ||
{self.gt_plane_mse},{self.plane_fit_rmse},{self.max_error},{self.min_plane_error},\ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change gt_plane_mse
-> gt_plane abs error
in the header
|
1b9c4e1
to
57a7a9f
Compare
d2afe30
to
f1cb2a8
Compare
…ments into daxoft/depth_test
No description provided.