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

Feat: deletion of tfs via flatbuffers service #404

Merged
merged 8 commits into from
Aug 7, 2024

Conversation

mmeijerdfki
Copy link
Contributor

@mmeijerdfki mmeijerdfki commented Aug 6, 2024

This PR adds the ability for a client to request the deletion of a range of TFs by specifying timespan, parent frame and child frame. Refer to seerep_com/README.md and examples/python/gRPC/tf/gRPC_fb_deleteTfs.py for more information.
The feature in combination with TransferTransformStamped effectively enables the altering of TFs.

Changes and discussion points:

  • The spatial index is recreated whenever a stream of TF deletion requests has been sent and TFs were removed
  • The complete data of the TFs is read and rewritten to HDF5
  • A corresponding example and integration test was created
  • Minor simplification of the createQuery interface of fb_helper: builder.CreateString() on uuids is now done in the function
  • gRPC_fb_sendImages takes a frame_id and timestamps for its header as parameters to be used for TF tests
  • small fix in fb_to_dict.py, when variables weren't set
  • variations for the call_get_images_fb in service_manager.py to directly get a python dict
  • wrapper functions to the canTransform and transformAABB calls such that DatasetIndexable can be used directly

@mmeijerdfki mmeijerdfki linked an issue Aug 6, 2024 that may be closed by this pull request
@mmeijerdfki mmeijerdfki merged commit f210a45 into main Aug 7, 2024
8 checks passed
@mmeijerdfki mmeijerdfki deleted the feat/rm-tf-fb-service branch August 7, 2024 13:06
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

Successfully merging this pull request may close these issues.

Update the tf
2 participants