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

Samples: Fix issues in ZividBenchmark #268

Merged
merged 1 commit into from
Dec 17, 2024

Conversation

csu-bot-zivid
Copy link
Contributor

@csu-bot-zivid csu-bot-zivid commented Dec 13, 2024

There was some incorrect usage of the new combined settings object.

In addition to explicitly calling capture2D and capture3D, we
explicitly call the settings by a more descriptive name.

Finally, a few minor spell checks.

@csu-bot-zivid csu-bot-zivid force-pushed the 2024-12-13-update-cpp-samples branch from e0a1201 to a0e7805 Compare December 13, 2024 13:05
There was some incorrect usage of the new combined settings object.

In addition to explicitly calling capture2D and capture3D, we
explicitly call the settings by a more descriptive name.

Finally, a few minor spell checks.
@SatjaSivcev SatjaSivcev force-pushed the 2024-12-13-update-cpp-samples branch from a0e7805 to 33d7eba Compare December 16, 2024 11:44
@SatjaSivcev SatjaSivcev changed the title Samples: Automatic updates to public repository Samples: Fix issues in ZividBenchmark Dec 16, 2024
Copy link
Contributor

@SatjaSivcev SatjaSivcev left a comment

Choose a reason for hiding this comment

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

LGTM

@SatjaSivcev SatjaSivcev merged commit 5ebc020 into master Dec 17, 2024
2 checks passed
@SatjaSivcev SatjaSivcev deleted the 2024-12-13-update-cpp-samples branch December 17, 2024 07:15
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