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

feat(lidar_transfusion): update TransFusion-L model (#7890) #1479

Closed
wants to merge 1 commit into from

Conversation

scepter914
Copy link

Description

Related links

Parent Issue:

  • Link

How was this PR tested?

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

…on#7890)

* add num_proposals

Signed-off-by: scepter914 <[email protected]>

* fix config

Signed-off-by: scepter914 <[email protected]>

* update README

Signed-off-by: scepter914 <[email protected]>

---------

Signed-off-by: scepter914 <[email protected]>
Copy link

Copy link

@amadeuszsz amadeuszsz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@knzo25
Copy link

knzo25 commented Sep 11, 2024

@scepter914
Will this PR be merged ? (otherwise, it may be better to close it or put it as a draft)

@scepter914
Copy link
Author

@knzo25

Yes.

I'm sorry that I didn't write status.
This PR waits for integration test of SI team for now, so after that I'll merge.

@knzo25 knzo25 removed their request for review October 9, 2024 07:33
@shmpwk
Copy link
Collaborator

shmpwk commented Oct 10, 2024

@scepter914 @amadeuszsz @knzo25
We are so sorry that we could't apply this transfusion-l model in beta/v0.29.0 (pilot-auto.xx1 v0.46.* release)
Let's try this in xx1 v0.47.0 release instead 🙏

@shmpwk shmpwk closed this Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants