Skip to content

Partial frame notif acc to kpi - #10147

Merged
ev-mp merged 5 commits into
realsenseai:developmentfrom
remibettan:partial-frame-notif-acc-to-kpi
Jun 21, 2022
Merged

Partial frame notif acc to kpi#10147
ev-mp merged 5 commits into
realsenseai:developmentfrom
remibettan:partial-frame-notif-acc-to-kpi

Conversation

@remibettan

Copy link
Copy Markdown
Contributor

Tracked by: DSO-17850

@remibettan
remibettan requested a review from ev-mp January 11, 2022 13:24
Comment thread src/linux/backend-v4l2.cpp Outdated
Comment thread src/linux/backend-v4l2.h Outdated
Comment thread src/linux/backend-v4l2.h Outdated
Comment thread src/linux/backend-v4l2.h Outdated
Comment thread src/linux/backend-v4l2.h Outdated
Comment thread src/linux/backend-v4l2.h Outdated
Comment thread src/linux/backend-v4l2.h
Comment thread src/linux/backend-v4l2.h Outdated
Comment thread src/linux/backend-v4l2.cpp
Comment thread src/linux/backend-v4l2.cpp Outdated

@Nir-Az Nir-Az left a comment

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.

LGTM

Comment thread src/linux/backend-v4l2.h
double _kpi_frames_drops_pct;
};

class v4l_uvc_device : public uvc_device, public v4l_uvc_interface

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Check with kernel 5+: the 'v4l_uvc_device' class may not apply there

Comment thread src/linux/backend-v4l2.cpp Outdated
Comment thread src/linux/backend-v4l2.h Outdated
Comment thread src/linux/backend-v4l2.cpp Outdated
@maloel

maloel commented Jun 20, 2022

Copy link
Copy Markdown
Contributor

@remibettan please work with @ev-mp
This PR is needed but the Jira ticket is marked as Implemented for some reason...

@ev-mp
ev-mp merged commit a6357eb into realsenseai:development Jun 21, 2022
@remibettan
remibettan deleted the partial-frame-notif-acc-to-kpi branch July 29, 2025 14:10
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