Add Prediction Interface and linear position prediction model to Object Memory
Merged
Add Prediction Interface and linear position prediction model to Object Memory
linear-prediction
into
master
1 unresolved thread
1 unresolved thread
Closes #91 (closed), #96 (closed).
Add linear position prediction to ObjectMemory Segment.
Edited by Rainer Kartmann
Merge request reports
Activity
added hiwi module::armem labels
added 5 commits
Toggle commit list- Resolved by Philip Scherer
- Resolved by Philip Scherer
- Resolved by Philip Scherer
Some things I noticed while working on this MR:
- Using
std::stringstream
s for error messages in BOs is not as great as we thought because it makes the BOs non-copyable, so you can't use them in vectors anymore. - Last time, we discussed that predictions should use MemoryIDs that are resolved to the snapshot level. However, I think that object pose predictions require instance indices for disambiguation. In particular, the current code only works if every entity in the instance segment only has a single instance per snapshot. If it has multiple object instances, the predictions will mix up the different objects' poses. I'm not quite sure how to resolve this since
ObjectID
s support arbitrary strings as instance names - armem only supports integers, so the conversion would be a little iffy.
- Using
- Resolved by Rainer Kartmann
Follow up:
Refactor prediction code from
source/RobotAPI/libraries/armem_objects/server/instance/Segment.cpp
into the library RobotAPIArmarXObjects (i.e. something like:ObjectPose predict(vector<ObjectPose> poses, time)
)
mentioned in issue #96 (closed)
added 47 commits
-
ea64a429...60f6e344 - 44 commits from branch
master
- 60d248e5 - Make PredictionResult error messages into strings
- a7eb5e0e - Merge branch 'master' into linear-prediction
- 238744af - Use entity name as object ID
Toggle commit list-
ea64a429...60f6e344 - 44 commits from branch
added 1 commit
- 96ade75c - Improve fetching prediction data in SegmentAdapter
added 1 commit
- 854e838b - ObjectMemory Visu: Predictions with fewer locks
added 1 commit
- 283fb965 - ObjectMemory Visu: Predictions with fewer locks
added 1 commit
- 90bbd193 - Do not use armem prediction settings in ArmarXObjects lib
mentioned in commit 87eff69d
Please register or sign in to reply