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

add API function unload_tracking_module. #3339

Merged
merged 2 commits into from
Mar 2, 2019

Conversation

doronhi
Copy link
Contributor

@doronhi doronhi commented Feb 27, 2019

Used to cleam _tm_context object if tracking module is not used, so it will be available for other applications.
Can be used prior to query_devices() to prevent connecting to tracking modules or afterwards to release all tracking modules.

…ject if tracking module is not used, so it will be available for other applications.
@dorodnic dorodnic added libtm T260 series Intel® T265 library labels Feb 27, 2019
@dorodnic
Copy link
Contributor

Related to #2924

* query_devices locks the tracking module.
* If the tracking module device is should be removed so that other applications could find it.
*/
void rs2_context_unload_tracking_module(rs2_context* ctx, rs2_error** error);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to add function name to realsense.def file

@@ -63,6 +63,13 @@ rs2_device* rs2_context_add_device(rs2_context* ctx, const char* file, rs2_error
*/
void rs2_context_remove_device(rs2_context* ctx, const char* file, rs2_error** error);

/**
* Removes tracking module.
* query_devices locks the tracking module.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The description of both the original issue and the solution is not clear - please review and adjust.

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

Successfully merging this pull request may close these issues.

3 participants