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

Added automatic duplex setting of Decklink device (Decklink SDK update) #1415

Closed
wants to merge 4 commits into from

Conversation

pkeuter
Copy link
Collaborator

@pkeuter pkeuter commented Feb 7, 2022

I often notice that the Decklink duplex settings have been reset after doing a Windows update. When this happens, it yields some inconsistent behaviour.

To handle this, I've added a configuration setting which provides a way to consistently set the duplex-behavior of the specified channel e.g.:

<decklink>
  <device>1</device>
  <duplex>full</duplex>
</decklink>

Full is a connected output (SDIx&SDIy). Other options are half which is just SDIx. Setting nothing keeps the current behavior (leave the setting as is).

Because this functionality was not available in the current SDK provided with CasparCG which is quite old, I have also upgraded this.

There is some functionality change in the DoesSupportVideoMode-function which I've modified but as far as I can see this has no implications other than a slightly changed log-line.

@Julusian
Copy link
Member

Julusian commented Feb 7, 2022

What is the oldest sdk version that supports this property?
I think that users need to be using drivers the same version or newer than the version of the sdk we use, so it would be nice to not use the absolute latest if possible

@pkeuter
Copy link
Collaborator Author

pkeuter commented Feb 7, 2022

I need to look that up. You are right by the way, you need a driver version equal to or higher than the SDK used.

My time is limited so I'll get back on that version (and likely modified PR) later.

@pkeuter
Copy link
Collaborator Author

pkeuter commented Feb 14, 2022

superseded by #1416

@pkeuter pkeuter closed this Feb 14, 2022
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.

None yet

2 participants