Skip to content

D457 Segfault when calling get_motion_intrinsics - #12042

Merged
Nir-Az merged 10 commits into
realsenseai:developmentfrom
Tamir91:D457-segfault-error
Aug 8, 2023
Merged

D457 Segfault when calling get_motion_intrinsics#12042
Nir-Az merged 10 commits into
realsenseai:developmentfrom
Tamir91:D457-segfault-error

Conversation

@Tamir91

@Tamir91 Tamir91 commented Jul 26, 2023

Copy link
Copy Markdown
Contributor

Tracked on [DSO-19103]
Tracked on [LRS-845]

@Nir-Az

Nir-Az commented Jul 26, 2023

Copy link
Copy Markdown
Collaborator

Nice catch!, as discussed, please add a unit test for each(D400) and later on PR with each(D500) on the relevant branch

@Tamir91
Tamir91 force-pushed the D457-segfault-error branch from bc9fef8 to e9775c8 Compare August 7, 2023 09:33
Comment thread unit-tests/live/intrinsics/test-motion-intrinsics.py
@Tamir91
Tamir91 requested a review from Nir-Az August 7, 2023 11:23
@Nir-Az

Nir-Az commented Aug 7, 2023

Copy link
Copy Markdown
Collaborator

Please remove this file you added
unit-tests/.idea/workspace.xml

Comment thread unit-tests/live/intrinsics/test-motion-intrinsics.py Outdated
@@ -0,0 +1,33 @@
# License: Apache 2.0. See LICENSE file in root directory.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please rename the file to test-motion.py
Currently when it runs it looks like this
intrinsics-motion-intrinsics

@Tamir91 Tamir91 Aug 7, 2023

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The test was renamed.

Comment thread unit-tests/live/intrinsics/test-motion-intrinsics.py Outdated
Comment thread unit-tests/live/intrinsics/test-motion-intrinsics.py Outdated
Comment thread unit-tests/live/intrinsics/test-motion.py
@Nir-Az
Nir-Az merged commit 74c2dad into realsenseai:development Aug 8, 2023
@Tamir91
Tamir91 deleted the D457-segfault-error branch December 24, 2024 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants