VCS 1.13.4 released
Prompted by a couple reports, here's a new bugfix release of VCS. It most importantly fixes an incompatibility with Bash 5.0 which made the script put all the captures in a single row.
A couple minor bugs are also fixed, where using mawk would produce some errors and print the wrong file sizes, and the evasion offsets (which are used to avoid blank screengrabs) weren't really being used.
See the full changelog for more details.
- vcs 1.13.4
- vcs-1.13.4.gz (script only)
or
vcs-1.13.4.tar.gz (script, sample configuration, manpage and profiles) - deb, rpm, PKGBUILD & xz
- as usual at p.outlyer.net/vcs
I appreciate it, Thank you so much!!
I was looking for a similar tool for some time.
I had to modify it a bit according to my needs.
It was easy with almost zero BASH knowledge as the script is very well-organized.
Thank you, tip granted!
Thank you! I did my best to keep it organized and documented since it's a fairly complex and big script, I'm glad that was useful for someone else. If you think your modifications can be of use to others I'm always open to suggestions and patches.
Hello Toni,
sorry i just saw your reply now. 🙁
I modified script behavior if output file exists.
I use another bash script to invoke VCS for every video file on a folder recursively and for me it was handy to add an option to keep already existing output to reduce overall runtime.
I send you the modified script via e-mail and you can decide if it is useful or not.
Also i failed to find out where to add jpeg quality settings to reduce jpeg output sheet size. I think this feature would be useful as well.
Thank you again for this great – and free – tool! 🙂
Thank your for your suggestion and submission.
As for JPEG quality, while there's no specific command-line option to set it, it can be set through command-line overrides or configuration files (see https://p.outlyer.net/vcs/docs/conf_files)
To set it from the command-line use, e.g.:
$ vcs -O quality=85 …
Wow, i did not notice this feature. Thank you for quick response! :bow:
Excellent! 🙂
I am happy again, that I now can continue using the -S option, and that the Footer has finally returned.
However, some new thing seems to have entered the realm of vcs: -T pc3
I replaced the line "if [[ 8 = "$(tput -T pc3 colors)" ]]; then" with the line "if [[ 8 = "$(tput colors)" ]]; then" and colors returned. Before the change I always got the error: "tput: unknown terminal "pc3""
The system I run the vcs script on is old, very old, so it has no knowledge of updated Terminal versions, if that might be the case.
The code though, seems to me that you only need to compare the number 8 with how many colors the Terminal actually supports, and in my case that's 8. What the -T option has got to do with anything, I am not sure. Something more modern perhaps? Or has it something to do with your system? Personally I could live without the colors. They are not necessary for the functionality of the application.
Anyway, great work! 🙂
Thank you for your input. Getting tput to produce consistent output across different OSes is harder than I'd like and it involved some hair pulling when dealing with BSDs, so that may be the source of the pc3 type, I honestly can't remember right now.
Nonetheless, I'm taking note of this for further testing.
Best Regards.
I used vcs-1.13.2 for a long time, but after my system upgrade the media info on the upper right side shifted out from the contact-sheet.
I upgraded to vcs-1.13.4 but it would not run as a script starting from my filebrowser (this is handy to right-click a file, start vcs and get an easy contactsheet for the file)
As I found out the problem is the tput detection:
"stty: 'standard input': Inappropriate ioctl for device"
"tput: No value for $TERM and no -T specified"
I went back to the old detection, pasted/replaced it into the new script and now it is working again:
I thought I let you know. Perhaps you can improve your new detection to run without exiting if no Terminal is found.
It definitely can be improved. As I replied above I ended up with some solution that seemed to work across systems at the time but it wasn't that solid, obviously.
Thank you for your feedback, I'm taking note to refine it further.
It seems the version 1.13.4 jumps over the function that was there in earlier versions (if memory serves me), the function that corrected timing for "decimal seconds". If a thumbnail landed on, say 05:24.100, the script corrected that to 05:25.00. But now it is left as an exercise to the reader to figure out what that 05:24.100 means. 🙂
Not a big problem, it just looks a little weird with three digits instead of the "normal" two. I put that in quotes since nothing in 2020 so far has been "normal".
Live well and prosper, and breathe carefully when you're out there. 😉
I'm not sure what you mean, I can't remember having such correction, timestamps are explicitly printed with two decimals. I'm getting two decimals on my system, maybe it is some weird interaction with a newer version of some package?
There was no rounding either as you seem to imply, the closest I can think of is when using MPlayer as capturer seeking doesn't have such precision. You can force using MPlayer with the -M switch if you prefer that behaviour.
output file is in 64b bit depth. how can I set it to lower bit such as 24?
I've haven't encountered such problem yet, and it sounds fairly weird.
You may try editing the script and adding
-depth 8to the final bigconvertcommand (on line 4135 in version 1.13.4) and see if that does the trick.Would it be possible to somehow specify to only screenshot half of the video image (for 180 side-by-side VR videos)?
Or would this require new features to be added to the VCS?
Not at this moment, yes it would need changes. I'm taking note since it might be useful, thank you for asking.
Hi,
I was getting:
Detected video length can't be reached. Safe measuring enabled.
On a video with a length that was not an integer (it has N.51 seconds).
The script did look at other length and produced a screensheet but I figured that a fractional second is probably not that useful so getting rid of it would get rid of the warning.
So I slightly modified your script to get rid of it and it worked.
I got some captured image off by a couple millisecond between the original version and the no fractional second version when there are only a few captured images, as is to be expected, but for me it worked well.
In case you are interested, the diff for it is:
Reaching the end shouldn't be affected by fractional seconds, so this may hint at other problems. I'm taking note of this, thank you for your contribution!
I'm having a problem with 1.13.4. I'm using ffmpeg 4.3.2 (release just before most current one). VCS is putting the bitrate in place of the width in the dimension field.
https://imgur.com/rdvfl8i
I guess they've changed the format of the output. Thank you, I'll look into it as soon as I have some spare time.
I can't reproduce, my version of ffmpeg 4.3.2 (and also 4.4) generates a proper header.
Can you copy the output of running
vcs -DD?(Output from a different video but has the same problem)
Video Contact Sheet *NIX v1.13.4, (c) 2007-2019 Toni Corvera Testing internal consistency... Passed test (seq replacement): consistent result Passed test (Identity): 'rmultiply 1,1 1'. Passed test (Rounding): 'rmultiply 16/9,1 2'. Passed test (Rounding): 'rmultiply 4/3 1'. Passed test (Commutative property): 'rmultiply 1,16/9 2'. Passed test (Alternate syntax): 'rmultiply 1.7 2'. Passed test (unnamed): 'ceilmultiply 1,1 1'. Passed test (unnamed): 'ceilmultiply 4/3 2'. Passed test (unnamed): 'ceilmultiply 7,1/2 4'. Passed test (Alternative syntax): 'ceilmultiply 7/2 4'. Passed test (Commutative property): 'ceilmultiply 1/2,7 4'. Passed test (Padding): 'pad 10 0 0000000000'. Passed test (Unneeded padding): 'pad 1 20 20'. Passed test (Floating point padding): 'pad 5 23.3 023.3'. Passed test (DVD AR Guess): 'guess_aspect 720 576 4/3'. Couldn't guess aspect ratio. Passed test (Unsupported Wide AR Guess): 'guess_aspect 1024 576 1024/576'. Passed test (lowercase conversion): 'tolower ABC abc'. Passed test (Integer pythagorean theorem): 'pyth_th 4 3 5'. Passed test (FP pythagorean theorem): 'pyth_th 16 9 18.35755975068581946630'. Passed test (Hours parsing): 'get_interval 2h 7200'. Passed test (Minutes parsing): 'get_interval 2m 120'. Passed test (Seconds parsing): 'get_interval 30S 30'. Passed test (Milliseconds parsing): 'get_interval .30 .30'. Passed test (Parsing with leading zeroes): 'get_interval 09h010m09s1 33010'. Passed test (Parsing shorthand): 'get_interval 0400 400'. Passed test (Repeated minutes parsing): 'get_interval 30m30m1h 7200'. Passed test (Leading zeroes in file size (GiB)): 'get_pretty_size 1127428915 1.05 GiB'. Passed test (Leading zeroes in file size (MiB)): 'get_pretty_size 1132462 1.08 MiB'. Passed test (Leading zeroes in file size (KiB)): 'get_pretty_size 1116 1.09 KiB'. Passed test (Pretty-printed file size (GiB)): 'get_pretty_size 1889785610 1.76 GiB'. Passed test (Pretty-printed file size (MiB)): 'get_pretty_size 762650296 727.32 MiB'. Passed test (Pretty-printed file size (KiB)): 'get_pretty_size 524810 512.51 KiB'. Passed test (FP test): 'fptest 3 -eq 3; returns 0'. Passed test (FP test): 'fptest 3.2 -gt 1; returns 0'. Passed test (FP test): 'fptest 1/2 -le 2/3; returns 0'. Passed test (FP test): 'fptest 6.34 -gt 6.34; returns 1'. Passed test (FP -logical- test): 'fptest (1>0) -eq 1; returns 0'. Passed test (Numeric recognition): 'is_number 3; returns 0'. Passed test (Quoted numeric recognition): 'is_number '3'; returns 1'. Passed test (Non-numeric recognition): 'is_number 3.3; returns 1'. Passed test (Float recognition): 'is_float 3.33; returns 0'. Passed test (Float recognition): 'is_float 3; returns 0'. Passed test (Non-float recognition): 'is_float 1/3; returns 1'. Passed test (Fraction recognition): 'is_fraction 1/1; returns 0'. Passed test (Non-fraction recognition): 'is_fraction 1; returns 1'. Passed test (Non-fraction recognition): 'is_fraction 1.1; returns 1'. Passed test (Positive recognition): 'is_pos_or_percent 33; returns 0'. Passed test (Percent recognition): 'is_pos_or_percent 33%; returns 0'. Passed test (Percent recognition): 'is_pos_or_percent 4/4%; returns 1'. Passed test (Percent recognition): 'is_pos_or_percent %; returns 1'. All tests passed Command line: /home/user/bin/vcs-1.13.4.bash -DD Walking_Polperro_Cornwall_UK_2021_Village_and_Harbour_Tour-B0UWmWMubLc.mp4 -U0 -c 3 -o Walking_Polperro_Cornwall_UK_2021_Village_and_Harbour_Tour-B0UWmWMubLc.mp4.jpg [TRACE]: pick_tools [TRACE]: set_capturer ffmpeg 0 [ svn $Rev: 688 $ ] === Setup === GETOPT: /usr/bin/getopt MPLAYER: FFMPEG: /usr/bin/ffmpeg AWK: /usr/bin/gawk sed: /bin/sed POSIXLY_CORRECT: {not set} Capturers (av.): [ ffmpeg ] Identif. (av.): [ ffmpeg ] Capturer: ffmpeg Chosen capturer: {default} Filterchain: [ filt_resize filt_apply_stamp filt_softshadow ] Safe step: 0.5 Blank evasion: Enabled (-5 +5 -10 +10 -30 +30) === Versions === Bash: 5.1.4(1)-release Getopt: getopt from util-linux 2.36.1 AWK: GNU Awk 5.1.0, API: 3.0 (GNU MPFR 4.1.0, GNU MP 6.2.1) sed: sed (GNU sed) 4.7 MPlayer: FFMpeg: ffmpeg version 4.3.2-0+deb11u2 Copyright (c) 2000-2021 the FFmpeg developers / libavcodec 58. 91.100 / 58. 91.100 ImageMagick: ImageMagick 6.9.11-60 Q16 x86_64 2021-01-25 https://imagemagick.org LSB Description: Debian GNU/Linux 11 (bullseye)Thank you, I'll see if I can I figure out what's the issue and let you know either way
Hey,
I'm using VCS via Homebrew Install (https://formulae.brew.sh/formula/vcs#default) and since the last update of imagemagick (?) I get dozens of error messages:
convert: Expected operator at '.mean*100' @ error/fx.c/GetOperator/2408
Can someone help please?
Sorry for the delay in aproving the message, I missed it. I can confirm the latest version of ImageMagick produces such warnings, I'll look into it.
Based on what I can see the error disables "blank frame evasion" but shouldn't have any other side effect.
In the mean time until I publish an update the trunk version should be working properly: https://svn.outlyer.net/svn/pub/video-contact-sheet/trunk/dist/vcs
With the new script I'm getting issues the following error:
No compatible version of getopt in path, can't continue.
Enhanced getopt (i.e. GNU getopt) is required
That's weird, I can't think of any recent change that would trigger that. Did it work in previous versions on the same system?