Skip to content

Conversation

@sickgreg
Copy link
Contributor

@sickgreg sickgreg commented Jan 2, 2026

No description provided.

PkgConfig::gstreamer
PkgConfig::gstreamer-app
PkgConfig::gstnet
PkgConfig::gio
Copy link
Collaborator

Choose a reason for hiding this comment

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

We need this also in the test build down below

run.txt Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we remove this ? looks like a local test script to me

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes and anything else we don't need

Copy link
Collaborator

Choose a reason for hiding this comment

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

Just a comment: While this might be useful, doing a cross compile is so much faster imho.

This already has the LUT correction dependencies which are not useful in context of this pr.
There is no harm in merging this though. Consider breaking up features in several logical pr if possible for the future.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah that's not meant to be in there. LLM brought it across from something I referenced in a prompt. Good thing you're on to it. Happy for you to tidy it up accordingly. I know what I want... Just maybe not how to get there

@henkwiedig
Copy link
Collaborator

Flew two packs indoors, seems to do what it is supposed to do.

@henkwiedig
Copy link
Collaborator

Added suggestions in https://github.com/sickgreg/PixelPilot_rk/pull/2/files

@henkwiedig henkwiedig merged commit 9121fb3 into OpenIPC:master Jan 6, 2026
6 checks passed
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