-
Notifications
You must be signed in to change notification settings - Fork 38
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
V3.0 #68
Conversation
NDIlib_tally_t* NDI_tally; | ||
NDIlib_send_get_tally(this->pNDI_send, NDI_tally, 0); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Per the example in the SDK, the following is correct:
NDIlib_tally_t NDI_tally;
NDIlib_send_get_tally(this->pNDI_send, &NDI_tally, 0);
|
||
char pixelStatus; | ||
std::ofstream neopixel; | ||
|
||
if(this->isProgram()) | ||
std::cout << "PGM: " << NDI_tally->on_program << " PVW: " << NDI_tally->on_preview << std::endl; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be std::cout << "PGM: " << NDI_tally.on_program << " PVW: " << NDI_tally.on_preview << std::endl;
if(this->isProgram()) | ||
std::cout << "PGM: " << NDI_tally->on_program << " PVW: " << NDI_tally->on_preview << std::endl; | ||
|
||
if(NDI_tally->on_program) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be if(NDI_tally.on_program)
} | ||
else if (this->isPreview()) | ||
else if (NDI_tally->on_preview) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be else if (NDI_tally.on_preview)
I would have submitted a pull request, but my fork has other unrelated changes, and I haven't quite figured out how to cherry pick in git. |
Figured it out, I submitted a pull request. I confirmed it working on my RPI 4B (don't know about the neopixel stuff, since I don't have one). |
No description provided.