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

Update nvidia keys #1632

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

danlipsa
Copy link
Collaborator

@danlipsa danlipsa commented Jun 6, 2022

No description provided.

This was triggered if no tracks were created on the first frame.
…es-core-deref

Avoid dereferencing an end iterator in track_features_core
If not, there's at least one issue due to threading, namely that the
call to merge_tracks can ultimately call erase on
simple_track_set_implementation::data_, which can cause the call to
simple_track_set_implementation::tracks, which copies data_ and is
indirectly made by match_tracks, to copy data_ while elements are
actively being removed from it.  This manifests as empty shared_ptrs
in the return value of tracks that are dereferenced in
track_set_implementation::active_tracks to cause a segfault.

Commit 17c7936 appears to have broken
this.
…exhaustive-threading

Fix race condition in close_loops_exhaustive
@kwcvrobot
Copy link
Collaborator

@kwcvrobot
Copy link
Collaborator

@kwcvrobot
Copy link
Collaborator

@kwcvrobot
Copy link
Collaborator

@danlipsa
Copy link
Collaborator Author

@mleotta Please review. Without this, the docker does not build.

@mleotta mleotta self-requested a review June 10, 2022 17:37
Copy link
Member

@mleotta mleotta left a comment

Choose a reason for hiding this comment

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

@danlipsa can you put this change on the release branch and document it in the release notes? It looks like a bug fix.

@danlipsa danlipsa force-pushed the update-nvidia-keys branch from 930a90e to ad0be47 Compare June 13, 2022 13:10
@danlipsa danlipsa force-pushed the update-nvidia-keys branch from ad0be47 to 0c2ace6 Compare June 13, 2022 13:28
@danlipsa
Copy link
Collaborator Author

danlipsa commented Jun 13, 2022

@mleotta I based this change on the release branch and updated the release notes.

@kwcvrobot
Copy link
Collaborator

@kwcvrobot
Copy link
Collaborator

@kwcvrobot
Copy link
Collaborator

@kwcvrobot
Copy link
Collaborator

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.

4 participants