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

Bugfix + cimbar_recv, a test decoder app + ... #61

Merged
merged 10 commits into from
Mar 26, 2022
Merged

Bugfix + cimbar_recv, a test decoder app + ... #61

merged 10 commits into from
Mar 26, 2022

Conversation

sz3
Copy link
Owner

@sz3 sz3 commented Mar 26, 2022

  • the finder pattern extract logic was, simply, broken (and frustratingly, the test was wrong). This would lead to decodes succeeding normally from some camera angles, and being ineffective from others.
  • added cimbar_recv for linux (maybe other desktop OSs eventually). It is very barebones.
    • example command line: ./cimbar_recv -i /dev/video2 -o /tmp
    • notably, there's no progress indicator yet. Configuration options are minimal.
    • it's basically a toy right now. Eventually it may be more useful.
  • to help with autofocus (a bit), added a grid background to cimbar.js/.org

@sz3 sz3 merged commit 3cd792b into master Mar 26, 2022
@sz3 sz3 deleted the cimbar-recv branch March 26, 2022 01:14
std::swap(anchors[0], anchors[top_left]);
if (top_left != 1 and top_right != 1)
std::swap(anchors[1], anchors[2]);
anchors = {anchors[top_left], anchors[top_right], anchors[bottom_left]};
Copy link
Owner Author

Choose a reason for hiding this comment

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

Ground zero -- tried to get clever, wrote a bad test, convinced myself wrong logic was right... 😡

sz3 added a commit that referenced this pull request Mar 13, 2024
3cd792b Merge pull request #61 from sz3/cimbar-recv
395d9ea Wire up FPS
7f19118 Clean up headers a bit
8cbd4f3 Set FPS?
d3dc2bd Adding a background grid cimbarjs to (maybe) help with autofocus
97b118a Resize the gl viewport to match the window + ...
c3f26f7 Cleanup, things are working*
8069560 The test was wrong 😱
e1fdcc7 good lord
8c23f60 Reorder various things... still not working...
d5dc994 WIP for cimbar_recv

git-subtree-dir: app/src/cpp/libcimbar
git-subtree-split: 3cd792b
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.

1 participant