-
Notifications
You must be signed in to change notification settings - Fork 1k
support bevfusion infer #2700
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
support bevfusion infer #2700
Conversation
Thanks for your contribution! |
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.
有几个地方需要确认下。另外重要函数、方法的docstring和type hints写清楚,docstring风格尽量参考Google Python Style Guide。
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.
麻烦再看下这几个地方
return points[not_close] | ||
|
||
def filter_point_by_angle(self, points): | ||
if isinstance(points, np.ndarray): |
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.
docstring
support new inference for BEVFusion