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

Camera custom tuning #131

Merged
merged 4 commits into from
May 10, 2021
Merged

Camera custom tuning #131

merged 4 commits into from
May 10, 2021

Conversation

alex-luxonis
Copy link
Contributor

@alex-luxonis alex-luxonis commented May 6, 2021

Add an option to override the default camera IQ (Image Quality) tuning database.
This is used for all cameras in the system, and specified at pipeline level:
pipeline.setCameraTuningBlobPath("path/custom_tuning.bin")

Related PR: luxonis/depthai-shared#30

Internally it is stored as a pipeline asset (like an NN blob), and loaded over XLink, or flashed together with the pipeline.

Note: building such a tuning blob need to go through a request to Luxonis, as the tools required are not public.

@Luxonis-Brandon Luxonis-Brandon added the Gen2 Feature for or issue with Gen2 label May 6, 2021
void PipelineImpl::setCameraTuningBlobPath(const std::string& path) {
std::string assetKey = "camTuning";

std::ifstream blobStream(path, std::ios::in | std::ios::binary);
Copy link
Collaborator

Choose a reason for hiding this comment

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

std::ios::in is redundant, that's why it's ifstream == input fstream
https://en.cppreference.com/w/cpp/io/basic_ifstream/basic_ifstream

Copy link
Collaborator

@themarpe themarpe left a comment

Choose a reason for hiding this comment

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

Looks good - thanks!

@alex-luxonis alex-luxonis merged commit 4903823 into develop May 10, 2021
@alex-luxonis alex-luxonis deleted the camera_custom_tuning branch May 10, 2021 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Gen2 Feature for or issue with Gen2
Development

Successfully merging this pull request may close these issues.

None yet

4 participants