Julian Waller

This wont be anything specific to satellite, this must have broken for non satellite devices too?

One thing I am noticing is there is a lot of small bits of tidying done here, many of them styling things that we at nrk don't necessarily agree on....

We don't need a system status check, as it will explicitly crash at startup with a helpful error https://github.com/nrkno/sofie-core/blob/feat/system-store-path-as-env-var/meteor/server/coreSystem.ts#L54 https://github.com/nrkno/sofie-core/blob/feat/system-store-path-as-env-var/meteor/server/coreSystem.ts#L480-L482 This is consistent with how other environment variables are handled...

One question I do have is, suppose that I have a change which requires a large re-implementation of a method. Once done it would give the same api as before,...

Wouldn't re-adding it be a minor bump? Because it will be a 'new' method, and so won't break anything My one concern is that it feels like we could end...

does it work with webhid on other machines?

Yep, this is a known issue with both [node-hid](https://github.com/node-hid/node-hid) and [@julusian/jpeg-turbo](https://github.com/julusian/node-jpeg-turbo) For now, stick with Electron 20. Ill fix these issues when I can, but if you use any other...

Another reference: https://github.com/bitfocus/loupedeck-ct

You are correct, and dont forget that a ssrc id field has been added near the beginning of the commands too. I can confirm it is a cut and paste...

I have opened a PR to do an overhaul of this library which includes an alternate solution to this (#228) As it doesn't look like there are any maintainers around...