Kok Seang Tan

Results 11 comments of Kok Seang Tan

Sorry for replying the old topic, but i would like to post my solution for others since have been working on this for few days Hi, I found that it's...

@ktro2828 @scepter914 Would you mind sharing your feedback about it? Thanks

In addition, I believe it is also important if we would like to apply this repo into our internal repo

Hi @scepter914 @ktro2828 Thanks for confirming this, I will work on this since it's part of evaluation pipeline in AWML

Hi, in case it can help anyone Basically, `LiDARInstance3DBoxes` assume it works with `bottom_center` instead of `geometry_center`, so `bottom_center` means (length/2, width/2, 0) for the center position of a 3d...

Hi @technolojin I decided to create another PR after cleaning up my previous PR, and I have moved the [INTERNAL LINK](https://tier4.atlassian.net/browse/RT1-10252) here

> I would like to suggest to add an argument `use_short_range_detection` for clear switching. > > * autoware/universe/launch/tier4_perception_launch/launch/perception.launch.xml > > add option > > ``` > > > ... >...

> @KSeangTan How shall we solve this issue? > > ![Screenshot from 2025-05-29 14-23-24](https://private-user-images.githubusercontent.com/42434141/448692416-8703fcad-e14f-47cf-b9c7-748998bf669f.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDg0OTc2NDYsIm5iZiI6MTc0ODQ5NzM0NiwicGF0aCI6Ii80MjQzNDE0MS80NDg2OTI0MTYtODcwM2ZjYWQtZTE0Zi00N2NmLWI5YzctNzQ4OTk4YmY2NjlmLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTA1MjklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwNTI5VDA1NDIyNlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTkwN2YwNDA0ZjQyMDQxZDA1OWJmMGRmM2VmZmQ1YjUxNDQ0MzQ3NTZhYjg1ZTRlZjgyZDQ1YzdmYjBmZTljZjAmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.nS8kBQj2NjHvwEWVlCxlHMepcOBCXOEq090Sjy2QMaE) > > option 1. let ci accept the name `shortrange` option 2. rename `shortrange` to `short_range`...

> > > @KSeangTan How shall we solve this issue? > > > ![Screenshot from 2025-05-29 14-23-24](https://private-user-images.githubusercontent.com/42434141/448692416-8703fcad-e14f-47cf-b9c7-748998bf669f.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDg0OTc2NDYsIm5iZiI6MTc0ODQ5NzM0NiwicGF0aCI6Ii80MjQzNDE0MS80NDg2OTI0MTYtODcwM2ZjYWQtZTE0Zi00N2NmLWI5YzctNzQ4OTk4YmY2NjlmLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTA1MjklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwNTI5VDA1NDIyNlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTkwN2YwNDA0ZjQyMDQxZDA1OWJmMGRmM2VmZmQ1YjUxNDQ0MzQ3NTZhYjg1ZTRlZjgyZDQ1YzdmYjBmZTljZjAmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.nS8kBQj2NjHvwEWVlCxlHMepcOBCXOEq090Sjy2QMaE) > > > option 1. let ci accept the name `shortrange` option 2....

Hi @technolojin Thanks for the quick discussion As per discussion in person, I have made the [changes](https://github.com/autowarefoundation/autoware_universe/pull/10727/commits/2e6f848a37eff91028b0d64d16c5cec803914af1) that move `short_range_centerpoint` as a variant of centerpoint, and it will launch another...