Skip to content

feat: support for scaling modes#1566

Merged
Julusian merged 3 commits into
CasparCG:masterfrom
dimitry-ishenko-casparcg:aspect
Sep 27, 2024
Merged

feat: support for scaling modes#1566
Julusian merged 3 commits into
CasparCG:masterfrom
dimitry-ishenko-casparcg:aspect

Conversation

@dimitry-ishenko
Copy link
Copy Markdown
Contributor

Ported from nrkno#28 and added HFILL and VFILL modes.

Only image_producer for now. Will try to do the rest tomorrow.

Comment thread src/modules/image/producer/image_producer.cpp
@dimitry-ishenko
Copy link
Copy Markdown
Contributor Author

dimitry-ishenko commented Sep 20, 2024

Added ffmpeg_producer as well. The other two in your PR are the channel_producer and layer_producer, neither one of which exist currently. I believe they've been superseded by the route_producer?

I will mark it as ready to be pulled. If you see anything else, let me know.

@Julusian I can add a helper as you've suggested, but I can't figure out a good place to put it. If you do, let me know and I can refactor. But so far, it's only used in two places, so it may not be worth the trouble.

@dimitry-ishenko dimitry-ishenko marked this pull request as ready for review September 20, 2024 02:08
@dimitry-ishenko
Copy link
Copy Markdown
Contributor Author

If possible, I would like this to also be back-ported to the 2.4.x branch. I am trying to set up our new playout machine some time next week, and hopefully 2.4.1 will be out by then.

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.

2 participants