Skip to content

Ground Truth TF's slow after update #659

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

Open
dan-riley opened this issue Oct 10, 2020 · 4 comments · May be fixed by #759
Open

Ground Truth TF's slow after update #659

dan-riley opened this issue Oct 10, 2020 · 4 comments · May be fixed by #759
Labels
help wanted Extra attention is needed

Comments

@dan-riley
Copy link

Since updating to the latest Ignition and SubT versions (as detailed in the update page), enableGroundTruth is essentially broken. We have tried several different robot models, and they all publish extremely slow TF's (around 0.5-1hz it seems) and there seems to be some other massive cpu or memory usage happening. When running with ground truth off everything runs as it did before. I'm not sure which update caused the change, but we're seeing it on multiple machines.

@acschang
Copy link
Contributor

The issue is that the transform for ground truth was moved to the static TF topic and thus is bound by the static_update_frequency. When using ground truth for testing, I update the <static_update_frequency>1</static_update_frequency> parameter in cave_circuit.ign or the submitted model's spawner.rb file to be significantly higher.

Please let me know whether or not this works for you. In the future the ground truth transform should be moved off of the static TF topic.

@nkoenig
Copy link
Contributor

nkoenig commented Oct 19, 2020

@dan-riley , did @acschang answer solve your problem?

@nkoenig nkoenig added the help wanted Extra attention is needed label Oct 19, 2020
@dan-riley
Copy link
Author

@nkoenig Yes, this seems to work. We used 50hz. I agree that it should probably be moved off of the static TF.

@peci1 peci1 linked a pull request Jan 19, 2021 that will close this issue
@peci1
Copy link
Collaborator

peci1 commented Jan 19, 2021

Solution proposed in #759.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
4 participants