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

Transformer::setExtrapolationLimit is deprecated and does not do anything #6

Open
ruffsl opened this issue Feb 19, 2016 · 8 comments
Open

Comments

@ruffsl
Copy link

ruffsl commented Feb 19, 2016

Looks like as of Inigo, the setExtrapolationLimit is now deprecated.
Also seem like its not really recommended:
http://answers.ros.org/question/32727/tflistenersetextrapolationlimit-does-this-really-allow-forward-extrapolation/

@soulslicer
Copy link

I have no way of going around this. I have two sensors publishing their tf at different times and there doesnt seem to be a way to disable extrapolation or set limits

@ruffsl
Copy link
Author

ruffsl commented Aug 10, 2016

Hmm. Pinging tf master @tfoote , any suggestions?
Here is the line in question:

tfListener_.setExtrapolationLimit(ros::Duration(0.1));

@tfoote
Copy link

tfoote commented Aug 10, 2016

Why does extrapolation help coordinate sensors at two times? You should wait until you have valid transforms for the data before using them whether or not you have one source or multiple sources. A MessageFilter or filters will likely do what you need. You can use it programmatically to coordinate data from multiple sources internally instead of relying on just the incoming topics. (Set the topic to an unused topic and use the add method manuall.)

@romainreignier
Copy link

Because of that issue, the package cannot work on Kinetic, right?
I did not managed to make it merge 2 laser scans.

@tfoote
Copy link

tfoote commented Dec 16, 2016

You should update the code to wait for both transforms to be available before processing instead of just one.

@differkhun
Copy link

Hello?I wanna merge four scan topics from one lidar(ibeo),but I still have the problem:

Transformer::setExtrapolationLimit is deprecated and does not do anything

Any solutions there ?
Thanks for your reply~~

awesomebytes added a commit to uts-magic-lab/ira_laser_tools that referenced this issue Jun 20, 2017
#0  0xb29183ee in ?? () from /lib/libc.so.6
iralabdisco#1  0x080c85a2 in Eigen::internal::handmade_aligned_free(void*) ()
iralabdisco#2  0x080c85ef in Eigen::internal::aligned_free(void*) ()
iralabdisco#3  0x080cf4d8 in void Eigen::internal::conditional_aligned_free<true>(void*) ()
iralabdisco#4  0x080d5a6f in void Eigen::internal::conditional_aligned_delete_auto<float, true>(float*, unsigned int) ()
iralabdisco#5  0x080d350e in Eigen::DenseStorage<float, -1, -1, -1, 0>::~DenseStorage() ()
iralabdisco#6  0x080ce018 in Eigen::PlainObjectBase<Eigen::Matrix<float, -1, -1, 0, -1, -1> >::~PlainObjectBase() ()
iralabdisco#7  0x080ce030 in Eigen::Matrix<float, -1, -1, 0, -1, -1>::~Matrix() ()
iralabdisco#8  0x080c4a65 in LaserscanMerger::scanCallback(boost::shared_ptr<sensor_msgs::LaserScan_<std::allocator<void> > const> const&, std::string) ()
@EchChou
Copy link

EchChou commented Nov 7, 2019

I have the same problem. Someone can help please.
Thank you.

@pietrocolombo
Copy link
Contributor

which branch do you use?
use the kinetic branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants