Skip to content

Keyers and Macros #10

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Keyers and Macros #10

wants to merge 5 commits into from

Conversation

@adamtow
Copy link

@adamtow adamtow commented Jan 14, 2021

Added the following commands: KeOn, CKOn, CDsL, DskS, CDsF, CDsC, DskB, MAct, MPrp, _MAC, and _pin (revised with new message) to support turning on/off the USK and DSK, setting the fill/key source for the DSK, listing macros, and running macros.

@adamtow
Implemented the following commands: KeOn, CKOn, CDsL, DskS to set and…
73da449 
… get the status of the upstream and downstream keyers. Added test scripts to control the USK 1 and DSK 1 on M/E 0.
@adamtow
Change framesRemaining to remainingFrames in ChangeDownstreamKeyerOnA…
14bbe23 
…ir message. Add Config.MacroPro and Do.MacroAction commands. Added Macro test script.
@adamtow
ProductName configuration message.
ae1b300
@adamtow
ProductName test script.
ac129c1
@adamtow
Commands to set the fill and key source for the Downstream Keyers.
f6df3c2
@adamtow adamtow mentioned this pull request Jan 14, 2021
Closed
@Dev1an
Copy link
Owner

Dev1an commented Jan 16, 2021 •
edited
Loading

Thank you for the contribution!

What is the ProductName (_pin) revision? Did you find something new or something that did not work with ProductInfo?

@adamtow
Copy link
Author

adamtow commented Jan 16, 2021

Thank you for the contribution!

What is the ProductName (_pin) revision? Did you find something new or something that did not work with ProductInfo?

Your implementation of the _pin message was crashing when I tried running it against my ATEM Mini Pro. When I implemented it according to the older protocol reference listed on the Skaarhoj site, it seemed to work, but I didn’t want to overwrite what you had added based on the libAtem protocol reference.

@Dev1an
Copy link
Owner

Dev1an commented Jan 16, 2021

OK I will look into that.

@Dev1an Dev1an mentioned this pull request Jul 21, 2021
Closed
@mwfrink
Copy link

mwfrink commented Oct 2, 2021

Is there a plan to integrate this change into the master or development branch? I’d like to use the macro support but I also want the newer bug fixes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@adamtow @Dev1an @mwfrink