-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Corrupted Maps Being Exported #4593
Comments
I tested it a little further, and when the map comes out corrupted the T265 is actually in a bad state where you can't interact with it further until you unplug it and plug it back in. |
Thanks @JBBee, after a while I was able to reproduce the issue as well |
Hi @JBBee - are you still running into corrupted maps issue? |
Hi @RealSenseCustomerSupport actually we are still seeing corrupted maps being exported every so often in 2.30.0. Please see below. Steps to reproduce in the viewer:
Thanks. |
Thanks for the procedure @JBBee. While we try and duplicate this, could you provide the final map that fails to import? |
Interesting. Will try to reproduce tomorrow. |
Thanks @JBBee. I'm able to duplicate the issue with that map. |
o Double map size o Fix loading maps with a previously loaded map that hadn't relocalized IntelRealSense#4593
o Double map size o Fix loading maps with a previously loaded map that hadn't relocalized IntelRealSense#4593
@radfordi I can confirm the issue. I additionally noticed no relocation info on the third run, step 3. Import failed too in step 4. Currently cloning your repo. While running Cmake I got this trace:
Not sure if it is just cosmetic. Anyway. After build I can't make your version run. Starting realsense-viewer just gives this
Since your PR is not merged yet, I have to wait for it I guess. Not a problem. |
Thanks @radfordi I'll grab it once it's merged. |
@neilyoung, those message aren't related to T265. When running -- Info: REALSENSE_VERSION_STRING=2.30.0 so I'd guess you don't have my branch checked out. Try doing |
True. Works. No corrupted map anymore. Double size confirmed. I thought I would have filed an issue, but I can't actually find it: At least under Ubuntu there is no .raw extension on save added. So the file open dialog doesn't find it. It is just an annoyance, just so you know. It's getting better day by day :) Great PR. |
o Double map size o Fix loading maps with a previously loaded map that hadn't relocalized IntelRealSense#4593 o Improve time to relocalization
o Double map size o Fix loading maps with a previously loaded map that hadn't relocalized IntelRealSense#4593 o Improve time to relocalization
o Double map size o Fix loading maps with a previously loaded map that hadn't relocalized IntelRealSense#4593 o Improve time to relocalization
o Double map size o Fix loading maps with a previously loaded map that hadn't relocalized IntelRealSense#4593 o Improve time to relocalization
@radfordi 0.2.0.879 in 2.31.0 works great. I'm not seeing any corrupted maps and the relocalization is faster. Thanks! |
Thanks for following up @JBBee. |
Issue Description
I'm seeing an issue with 2.25.0 where it's possible to be unable to import a map that was previously exported by the T265.
To reproduce this in the viewer all I have to do is start and map an area, then stop and export that map. Then import the same file I exported and see if it was successful. If it was successful I run the device for a little with the imported map until it relocalizes. Then stop and repeat the steps from the beginning. After roughly five times the map import fails with the message below.
Invalid Value in rs2_import_localization_map(sensor:0000022AA6265BE0, lmap_blob:R, blob_size:849021):
import localization failed, map size 849021
Thanks.
The text was updated successfully, but these errors were encountered: