Skip to content

Conversation

angeldeejay
Copy link

Added new features:

  • Aspect ratio correction (solves ISSUE 35)
  • Auto start zoom

@BlankSourceCode
Copy link
Owner

@angeldeejay Thanks for the PR!

Would it be possible to (either automatically by grabbing info from OBS or manually via textboxes) allow different output aspect ratios rather than just a single toggle for 16:9? I'm pretty sure it would then also fix #31 and #30.

The auto-zoom toggle may be able to fix #36 if it saves/restores the state based on the scene but that seems like a bit of a stretch goal.

@angeldeejay
Copy link
Author

@angeldeejay Thanks for the PR!

Would it be possible to (either automatically by grabbing info from OBS or manually via textboxes) allow different output aspect ratios rather than just a single toggle for 16:9? I'm pretty sure it would then also fix #31 and #30.

The auto-zoom toggle may be able to fix #36 if it saves/restores the state based on the scene but that seems like a bit of a stretch goal.

The "auto-zoom" feature I've implemented, just starts the script with OBS (without using key shortcut or anything). I've found it useful in cases when you need the zoom always active. I t not pretends to fix #36.

About the aspect ratios feature: this is an initial workaround to maintain proportions on ultrawide screens. The future goal should use the original aspect ratio from the source/device the script is trying to zoom in and adjust according to it, but I'm not able to implement something with that complexity right now due to I'm busy. (Sorry 😔).

@nschurmann
Copy link

I do like the idea of having an auto_start feature, however I tried this PR and is not working. It seems that you need to change an option, any option, to actually zoom. Then it get's locked in. @angeldeejay

@cyear
Copy link

cyear commented Sep 22, 2024

增加新功能:

Wow, thanks a lot, it made my script work fine!

@angeldeejay
Copy link
Author

I've changed it, but I'm waiting to get some time to test it properly and update the branch

@cyear
Copy link

cyear commented Sep 28, 2024

I've changed it, but I'm waiting to get some time to test it properly and update the branch

May I ask if the issue of not restoring after scaling is resolved? (I have encountered this problem)?

@angeldeejay angeldeejay force-pushed the main branch 2 times, most recently from 886981a to 2b1d764 Compare October 6, 2024 00:33
@angeldeejay
Copy link
Author

angeldeejay commented Oct 6, 2024

A short follow up:

  • Regarding this:

    I've changed it, but I'm waiting to get some time to test it properly and update the branch

    May I ask if the issue of not restoring after scaling is resolved? (I have encountered this problem)?

    Please provide steps to reproduce the problem or open another issue. I'm using 2 monitors with 21:9 and 32:9 (aspect ratios) and have no issues with any of them.

  • Regarding this:

    I do like the idea of having an auto_start feature, however I tried this PR and is not working. It seems that you need to change an option, any option, to actually zoom. Then it get's locked in. @angeldeejay

    I apologize for the confusion. What I meant with "auto start feature" was I want to get "locked in" auto-zoom always, not deactivate it. Even if I'm restarting OBS, it should keep auto zooming correctly.

    Maybe It is not the feature you are looking for, and we should rename the option label.

    My proposal to get working this feature in a way that can be deactivated is to add logic to keep synced auto-lock and activation status. In this way, you can deactivate auto zooming whenever you want, and if you shutdown OBS with auto zoom activated, restore auto zoom at the moment you start OBS again. Sadly, I'm not able to do this fix right now due to I'm currently involved in too many projects, sorry

Changes done for now:

  • Renamed "auto start" feature. Now is "lock-in". It has now a description explaining this as a tooltip.
  • Leave only featured changes against the main branch (keep code easy to review)

image

I will be in touch, for developing the features you are requesting soon

@angeldeejay angeldeejay force-pushed the main branch 4 times, most recently from fe8c64f to 13b9225 Compare October 6, 2024 00:39
@angeldeejay
Copy link
Author

angeldeejay commented Oct 6, 2024

@BlankSourceCode please open a feature request for the aspect ratio thing and tag me. I will be glad to review it with you and understand your request.

I can suggest providing a combo with common aspect ratios. If you let the user do whatever you want with the ratios, the script probably crashes OBS easily (I've really tried to do with textboxes)

@angeldeejay
Copy link
Author

angeldeejay commented Oct 6, 2024

Last update

  • Fixed bugfix on Auto Start start feature. It auto-zoom when OBS starts, but you can toggle zoom as always with keybinding
  • Aspect ratio param replaced in favor of Keep Shape param. When Keep Shape is enabled, the zoom window keeps aspect ratio according the aspect ratio has the source in canvas. This way, you can resize the source as you want, and the zoom window will "adapts" to the size of the source dynamically. Instead of adding parameters to setting up the aspect ratio, you should enable this option and resize the source to the aspect ratio you want. Easy peasy.

image

image

image

@angeldeejay angeldeejay force-pushed the main branch 2 times, most recently from fb5fe3a to da6f08b Compare October 6, 2024 06:22
@angeldeejay angeldeejay changed the title Improved auto start and force 16:9 aspect ratio features Improved auto start and respect aspect ratio features Oct 6, 2024
@Achuan-2
Copy link

Achuan-2 commented Dec 18, 2024

It would be even better if it could automatically decide whether to Zoom according to mouse event just like focusSee

@angeldeejay
Copy link
Author

It would be even better if it could automatically decide whether to Zoom according to mouse event just like focusSee

It actually does it that way. The screenshots above are just showing how the aspect ratio doesn't break the plugin with this update. Let's remember that contributors of this plugin are not focused on replicate FocuSee.

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.

5 participants