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

Remove low usage transfrom camera pipelines #4655

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

randhid
Copy link
Member

@randhid randhid commented Dec 26, 2024

The pipelines removed in this pr have been configured on fewer than five machines throughout Viam's entire lifetime. In the past year, 0 machines have been configured, and the pipelines have been removed.

The removed pipelines; unless otherwise specified the pipeline was configured on zero total machines.
"identity"
"overlay" - configured on 1 machine.
"depth-to-pretty" configured on 3 machines.
"depth-preprocess"
"depth-edges"
"undistort" - configured on 2 machines.
"segmentations" - not in docs

Corresponding docs PR viamrobotics/docs#3775

@viambot viambot added the safe to test This pull request is marked safe to test from a trusted zone label Dec 26, 2024
@randhid randhid requested a review from bhaney December 26, 2024 18:30
@randhid
Copy link
Member Author

randhid commented Dec 26, 2024

@bhaney won't merge this in until I ask my team if they find any of this useful - I'd also like to hear from your team if they think any of the deleted pipelines make sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe to test This pull request is marked safe to test from a trusted zone
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants