When testing I get Tried to reset when already active in the log.

Also result in a black screen after changing the profile.

Can you check that the merge from master was correct?

The Tried to reset when already active message was in the code before the PR. The resolution not changing when a new profile is chosen, while main output is active, is because we never responded to the OBS_FRONTEND_EVENT_PROFILE_CHANGING and OBS_FRONTEND_EVENT_PROFILE_CHANGED events. When these event types are added to the event callback for the plugin, and proper action is taken (calling main_output_deinit and main_output_init respectively), the main NDI output resolution changes, and as a bonus, we no longer get the Tried to reset when already active messages in the log.
This should when pulled this should also resolve: #1196 and #1195.

Tested on Mac. Seems to fit the bill.
Changed between profiles (1080P / 4K) > the output is recreated as expected.

In case of issues it solves :
#1147 : this will show an error in the log file, avoid a crash but still have the "output enabled" ticked.
This could be improved with #1148

#1195 : TBC

Seems with this PR, we are no longer outputting Preview scenes. The preview shows up as a NDI source, but it doesn't output anything. Will try and address it in PR 1148 I believe it is related to Preview and Main using the same OBS output.

The issue introduced by this PR is fixed by : #1237

Note for future self looking here : "try to keep the codebase between main output and preview output logic as close as possible!"