-
-
Notifications
You must be signed in to change notification settings - Fork 351
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
Preparing for 6.0.0 release #960
base: master
Are you sure you want to change the base?
Conversation
Also making Config code similar to `obs-websocket`'s code
...and not legacy NDI 4 `ndi_hwaccel`. According to their docs and forum posts there is supposedly no way to un-set this.
The current code either packages a .msi **or** generates a .zip. Many users prefer the zip (and it has the benefit of mitigating code signing warnings). This change always generates the .zip file. If `-BuildInstaller` is also specified then is **also** builds a .msi.
Also having urls launch a much more consistent way in output-settings, obs-ndi-filter, and obs-ndi-source. Commenting out currently unused obs_log(...) method. That method allocates and frees a formatting template every time it is called. All just to prefix the plugin module name in a `blog` output. obs-ndi `blog` already exhaustively prefix the module name, so this convenience function is unnecessary for now. If logging consistency ever gets out of hand then this could be uncommented and used.
Also making Config code similar to `obs-websocket`'s code
...and not legacy NDI 4 `ndi_hwaccel`. According to their docs and forum posts there is supposedly no way to un-set this.
The current code either packages a .msi **or** generates a .zip. Many users prefer the zip (and it has the benefit of mitigating code signing warnings). This change always generates the .zip file. If `-BuildInstaller` is also specified then is **also** builds a .msi.
Also having urls launch a much more consistent way in output-settings, obs-ndi-filter, and obs-ndi-source. Commenting out currently unused obs_log(...) method. That method allocates and frees a formatting template every time it is called. All just to prefix the plugin module name in a `blog` output. obs-ndi `blog` already exhaustively prefix the module name, so this convenience function is unnecessary for now. If logging consistency ever gets out of hand then this could be uncommented and used.
23d2148
to
36330b5
Compare
For anyone reviewing this: I am 100% aware that this PR is non-trivial (I hesitate to call it "large"; I have seen some crazy large PRs way bigger than this) and there is a reason why PRs should be small. Sorry for the below "rant"... I also am not a big fan of the branching strategy used here that is probably all my fault. My main gripe about our branching is that I am of the personal understanding/camp/belief/religion that everything on our
My understanding of how many Open Source projects on GitHub work (but perhaps not OBS Studio) that get lots of contributors is that most of them only ever release from master.
This tells me that master should always be in a shippable state. What this does not tell me or the uninformed reader/developer is how to actually release. Speaking of I could be wrong, I think that is how the OBS-NDI repo has historically worked before I got involved. When I started to get involved I unfortunately approached using this repo with a type of GitFlow mindset.
Simplified:
I am not going to get into any [religious] arguments about the pros or cons of GitFlow, I am just saying factually that I did try, right or wrong, to start using it in the OBS-NDI repo...and I kind-of regret that. I now believe that GitFlow does not work very well in an Open Source project where most code contributors don't know to follow GitFlow. GitFlow would only work if everyone follows it, like at a small team at a company where everyone understands their Way Of Work and there are no outside contributors. If someone on a GitFlow team starts branching feature PRs from One way to mitigate this in an Open Source project might be to make the People are going to be people. People are going to ignore GitFlow. So, this is a big long rant to say that I am open to future healthy constructive conversations about a better git branching/releasing strategy for OBS-NDI, but unless I hear any better ideas I am going to stop using a Perhaps we may eventually go to a more OBS-Studio type of flow (and what I also use at work) of also having a release branch (RB), and when that RB is created then the master is incremented to the next version that will be eventually be released in the future. Apologies again for the rant! :) |
b78f822
to
e29a7b0
Compare
Tbh, if we want ppl to submit to a specific branch we have to really make it clear. I have no specific preference, as long as it is clear / documented on how to contribute. |
Some update : I believe we should keep master for : whatever is the latest OFFICIAL tagged release. And push dev PR to the develop branch as initially expected. (to-do ; document in the dev wiki) The main reasons are : Other main argument :
And YES there might be time where we might have to update master branch even if this is not a full release. But this is an extrem use-case. |
Correcting typos & nitpicking.
@Trouffman please review my recent translation updates. |
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.
The proposed translation works as well - (seems to be more consistent with previous translations)
that was for the translation only - still have 2 files to review
NDIPlugin.SourceProps.LatencyMode="Latency Mode" | ||
NDIPlugin.SourceProps.LatencyMode.Normal="Normal (safest/recommended)" | ||
NDIPlugin.SourceProps.LatencyMode.Low="Low" | ||
NDIPlugin.SourceProps.LatencyMode.Lowest="Lowest (unbuffered; Alpha channel not supported)" | ||
NDIPlugin.SourceProps.Audio="Enable audio" | ||
NDIPlugin.SourceProps.PTZ="Pan Tilt Zoom" |
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 is confusing when displayed in the UI.
Should be : PTZ control
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.
Seem that the basics are working.
I have tested getting & sending on
- MacOS - Apple Silicon
Left to test : win10-11 & linux
PTZ could not be tested.
Stéphane Lepin stephane.lepin@gmail.com
toOBS-NDI Project obsndi@obsndiproject.com
--obs-ndi-verbose
command-line option to print out 'a' and 'v' for audio and video packets respectively