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

2D Ground Truth Sensor in 2021.3 doesn't detect pedestrians (It works in 2021.2) #1797

Open
zvika-waissbaum opened this issue Oct 7, 2021 · 16 comments
Labels
next release Feature/bug fix will be included in next release

Comments

@zvika-waissbaum
Copy link

No description provided.

@revati-naik
Copy link

@zvika-waissbaum Could you please elaborate on the issue? Are you not able to see the bounding boxes on the pedestrians when you try to visualize the sensor output in the simulator? Or you do not get the data on the topic?

@revati-naik revati-naik self-assigned this Oct 7, 2021
@zvika-waissbaum
Copy link
Author

zvika-waissbaum commented Oct 7, 2021 via email

@EricBoiseLGSVL
Copy link
Contributor

I'd make sure you have the latest 2d GT sensor. If you are using the old sensor with the new release code it will have many issues. Try to remove all local and cached sensors and make sure the sensor config is using the new sensor.
Also, be sure the sensor is not at origin, the vehicle will block detection of agents.

@zvika-waissbaum
Copy link
Author

It's the latest 2D GT sensor and it is placed above the vehicle (not in the origin).

@EricBoiseLGSVL
Copy link
Contributor

@zvika-waissbaum do you have any errors in the player.log? Any debug information would be helpful because we are not able to reproduce locally. Screen shots of the issue in play mode and of the sensor parameters would be great. Also the specs of the machine you are using with GPU/OS also will help.

@zvika-waissbaum
Copy link
Author

Screenshot from 2021-10-17 20-05-07
Screenshot from 2021-10-17 20-05-04

OS:Linux
GPU: Nvidia RTX2070

Don't have any errors in the unity environment.

@zvika-waissbaum
Copy link
Author

Screenshot from 2021-10-18 07-47-58

Maybe this is the issue?

@EricBoiseLGSVL
Copy link
Contributor

Yes, this is the issue. Did you spawn/despawn multiple times during a run? Our max values for pedestrians shouldn't hit the limit. Did you alter the source code to have more?

@zvika-waissbaum
Copy link
Author

No, I've spawn it once.
I'm attaching the python code.
It works in 2021.2 version.
Pedestrian.pdf

@EricBoiseLGSVL
Copy link
Contributor

We will look into this, thanks for posting. @kamkolak-lge do you need any more info to debug?

@EricBoiseLGSVL EricBoiseLGSVL added the bug Something isn't working label Oct 18, 2021
@kamkolak-lge
Copy link
Contributor

@zvika-waissbaum
Thank you for reporting the bug. It seems it's an issue with pedestrians spawned through Python API and does not affect simulation with random pedestrians.

If you could run a simulation with random pedestrians and 2D Ground Truth sensor to confirm the issue does not occur there, it would help us confirm the cause of this issue.

@zvika-waissbaum
Copy link
Author

Yes, you are correct.
It works fine with random pedestrians.

@kamkolak-lge
Copy link
Contributor

The issue has been fixed and will be included in the next release.

@EricBoiseLGSVL EricBoiseLGSVL added next release Feature/bug fix will be included in next release and removed bug Something isn't working labels Nov 19, 2021
@zvika-waissbaum
Copy link
Author

This bug really interferes with our development.
Can you upload a fix to the store with minor version change like 2021.3.1?

@EricBoiseLGSVL
Copy link
Contributor

We will be pushing a fix to master soon.

@peter0617ku
Copy link

@EricBoiseLGSVL @kamkolak-lge @zvika-waissbaum

Sorry for disturbing you!
I'm having this problem. Glad to see that your company has resolved this issue.
Because the Github changelog shows that this issue doesn't seem to be updated.
May I ask if LGSVL sunset will affect the update of this version?
If master won't be updated, maybe be able to tell us which files to change?

Thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next release Feature/bug fix will be included in next release
Projects
None yet
Development

No branches or pull requests

5 participants