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

Small fixes #631

Merged
merged 2 commits into from
Jan 15, 2024
Merged

Small fixes #631

merged 2 commits into from
Jan 15, 2024

Conversation

naushir
Copy link
Collaborator

@naushir naushir commented Jan 15, 2024

No description provided.

This might be needed by post processing stages to determine the sensor
resolution.

Signed-off-by: Naushir Patuck <[email protected]>
Allow the stage to run even if a low res stream is missing as it never
gets used by the stage.

Signed-off-by: Naushir Patuck <[email protected]>
@naushir naushir requested a review from davidplowman January 15, 2024 10:01
@davidplowman
Copy link
Collaborator

Are you happy to ignore the style checker? In which case it LGTM and am happy to merge...

@naushir
Copy link
Collaborator Author

naushir commented Jan 15, 2024

Yes, I think we can ignore the style checker here, it seems to love single line functions for some reason.

@davidplowman davidplowman merged commit 9d81389 into main Jan 15, 2024
9 of 10 checks passed
@naushir naushir deleted the fixes branch January 15, 2024 10:13
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.

2 participants