Skip to content
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

Add image sequence input (not fully merged back, Linux gui not added) #4461

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

sabvdf
Copy link

@sabvdf sabvdf commented Jul 19, 2022

Description of Change:

As discussed in Issue #2346
Using the image2 demuxer, adds the option for using an image sequence as input.
Option toggle is added to macOS and Windows GUI, not yet to Gtk version.

Tested on:

  • Windows 11
  • macOS 12

@galad87 galad87 added this to the 1.7.0 milestone Sep 12, 2022
@@ -33,6 +33,8 @@
"AudioSecondaryEncoderMode": true,
"AudioTrackSelectionBehavior": "first",
"ChapterMarkers": true,
"ImageSequence": false,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should these really be part of the preset keys?

Seems more like something that should be handled at source selection.

Ask for it up-front and maybe remember the last used framerate.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could be, I'm not too far into the project so I don't know about this kind of choice. Feel free to do with this what you think best, I think in this case I just took the quickest route to adding the functionality this way.

@galad87 galad87 removed this from the 1.7.0 milestone Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants