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

NOJIRA : add a 3rd attempt for failsafe video thumb extraction #1650

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

Conversation

gautiermichelin
Copy link
Contributor

NOJIRA :

  • correction of the 2nd : "// try again, with -ss 1" => the -ss 1 wasn't applied
  • add a 3rd failsafe video thumb extraction, without the "pad" indication inside the -vf, this secured me many cases where 1st and 2nd attempts gave nothing

yelto and others added 30 commits January 20, 2021 10:02
type_name_* prematurely set for listing sets by user
Localize select options for display settings
Quick fix for Set list by user headings
…ighlight-user-headings-to-help-read-the-listing

Set list by user: highlight user headings to help read the listing
using brackets doesn't work for PHP 8, throws deprecation warning in PHP 7.4
use [] instead of {} for getting character
Bring bracket back into its commented-out block
Localize the default text in required_field_marker, for instances where show_required_field_marker is set to 1.

The text to be translated would not be picked up if maintained verbatim  (that is, _t((Required)) would not be localized at runtime for some reason tied to the use of parenthesis). Not knowing the escape rules (if any), I've cowardly moved the string to square brackets, with _t([Required]) working as expected. Installs using the default string will notice a change from (Required) to [Required]. Hope it helps anyway.
gautiermichelin and others added 25 commits September 13, 2023 13:58
Update docx_results.php : missing field names in the docx export
Update docx_results.php : missing field names in the docx export
Removed duplicate relationship types from dacs_heritage.xml
added several new Nomisma classes for information service, refining c…
…rst level of places

Replacing : ca_places_locations_hierarchy_browser_hide_root
By : ca_places_hierarchy_browser_hide_root
Change ca_places_locations_hierarchy_browser_hide_root to hide the fi…
Corrected display format for Creator column
Fixing typo in global resourcespace information service settings variable.
…ToCorrectDocumentationOnProfiles

Correct the installer link to documentation about profiles
Enabled client-side caching in the IIIF service.
Update app.conf : logo for reports, svg is not valid for a docx export
Update app.conf : logo for reports, svg is not valid for a docx export
Fix: consistently translate "lot" to "Konvolut" or "Objektstapel" in German
NOJIRA :
- correction of the 2nd : "// try again, with -ss 1" => the -ss 1 wasn't applied
- add a 3rd failsafe video thumb extraction, without the "pad" indication inside the -vf, this secured me many cases where 1st and 2nd attempts gave nothing
@collectiveaccess
Copy link
Owner

What version(s) of ffmpeg is this tested with?

@gautiermichelin
Copy link
Contributor Author

ffmpeg version 4.3.8-0+deb11u1 ; debian 11.11
ffmpeg version 5.1.6-0+deb12u1 ; debian 12.8
ffmpeg version 7.1 ; mac os 14.1 & 15.1

@gautiermichelin
Copy link
Contributor Author

bump ?

@collectiveaccess
Copy link
Owner

This is against 1.7. Is that what you want?

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.